Skip to content

Commit d1f7214

Browse files
committed
Fix eon key publishing
We used to use the keyper internal eon index as identifier instead of the keyper set index.
1 parent 0d7a580 commit d1f7214

File tree

1 file changed

+14
-4
lines changed

1 file changed

+14
-4
lines changed

rolling-shutter/eonkeypublisher/eonkeypublisher.go

Lines changed: 14 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,7 @@ func (p *EonKeyPublisher) Publish(key keyper.EonPublicKey) {
8383
// set, unless the key is already confirmed or the keyper has already voted on it.
8484
func (p *EonKeyPublisher) publishIfResponsible(ctx context.Context, key keyper.EonPublicKey) {
8585
db := obskeyperdb.New(p.dbpool)
86-
keyperSet, err := db.GetKeyperSetByKeyperConfigIndex(ctx, int64(key.Eon))
86+
keyperSet, err := db.GetKeyperSetByKeyperConfigIndex(ctx, int64(key.KeyperConfigIndex))
8787
if err != nil {
8888
log.Error().
8989
Err(err).
@@ -121,11 +121,21 @@ func (p *EonKeyPublisher) publishOldKeys(ctx context.Context) {
121121
}
122122
dkgResult, err := shdb.DecodePureDKGResult(dkgResultDB.PureResult)
123123
if err != nil {
124-
err := errors.Wrapf(err, "failed to decode DKG result of eon %d", dkgResultDB.Eon)
125-
log.Error().Err(err).Msg("failed to publish old eon keys")
124+
log.Error().
125+
Err(err).
126+
Int64("eon", dkgResultDB.Eon).
127+
Msg("failed to decode DKG result to publish old eon key")
126128
continue
127129
}
128-
p.publish(ctx, dkgResult.PublicKey.Marshal(), dkgResult.Eon, dkgResult.Keyper)
130+
eon, err := db.GetEon(ctx, dkgResultDB.Eon)
131+
if err != nil {
132+
log.Error().
133+
Err(err).
134+
Int64("eon", dkgResultDB.Eon).
135+
Msg("failed to fetch eon to publish old eon public key")
136+
continue
137+
}
138+
p.publish(ctx, dkgResult.PublicKey.Marshal(), uint64(eon.KeyperConfigIndex), dkgResult.Keyper)
129139
}
130140
}
131141

0 commit comments

Comments
 (0)