Skip to content

Commit cef8e4d

Browse files
committed
Fix eon column in tx pointer table
Here, the eon refers to the keyper config index, not the keyper-internal eon. Note that the tx pointer is also read by other parties including validators, so using the proper eon (from Shuttermint) which is only meaningful in a keyper DKG context does not make sense. We should rename the message field in the spec at some point.
1 parent c3ef527 commit cef8e4d

File tree

1 file changed

+5
-5
lines changed

1 file changed

+5
-5
lines changed

rolling-shutter/keyperimpl/gnosis/newslot.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -211,16 +211,16 @@ func (kpr *Keyper) triggerDecryption(
211211
if err != nil {
212212
return errors.Wrapf(err, "failed to query eon for block number %d from db", nextBlock)
213213
}
214-
eon := eonStruct.Eon
214+
keyperConfigIndex := eonStruct.KeyperConfigIndex
215215

216-
txPointer, err := kpr.getTxPointer(ctx, eon, int64(slot), keyperSet.KeyperConfigIndex)
216+
txPointer, err := kpr.getTxPointer(ctx, keyperConfigIndex, int64(slot), keyperSet.KeyperConfigIndex)
217217
if err == errZeroTxPointerAge {
218218
log.Warn().
219219
Uint64("slot", slot).
220220
Int64("block-number", nextBlock).
221-
Int64("eon", eon).
221+
Int64("eon", keyperConfigIndex).
222222
Int64("tx-pointer", txPointer).
223-
Msg("skipping new block as tx pointer age is 0")
223+
Msg("skipping trigger as tx pointer age is 0")
224224
return nil
225225
} else if err != nil {
226226
return err
@@ -231,7 +231,7 @@ func (kpr *Keyper) triggerDecryption(
231231
return err
232232
}
233233
err = gnosisKeyperDB.SetCurrentDecryptionTrigger(ctx, gnosisdatabase.SetCurrentDecryptionTriggerParams{
234-
Eon: eon,
234+
Eon: keyperConfigIndex,
235235
Slot: int64(slot),
236236
TxPointer: txPointer,
237237
IdentitiesHash: computeIdentitiesHash(identityPreimages),

0 commit comments

Comments
 (0)