Skip to content

Conversation

@dai1975
Copy link

@dai1975 dai1975 commented Feb 3, 2026

elc-updater stores the signer address for client use.

Daisuke Kanda added 6 commits February 2, 2026 15:57
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
fix
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
@dai1975 dai1975 requested a review from siburu February 3, 2026 08:16
@dai1975 dai1975 changed the title et EKI signer address from grpc server get EKI signer address from grpc server Feb 4, 2026
Copy link
Contributor

@siburu siburu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dai1975 I have put comments.

relay/prover.go Outdated
Comment on lines 391 to 399
} else if len(messages) == 1 {
return []*elc.MsgAggregateMessages{
&elc.MsgAggregateMessages{
Signer: signers[0],
Messages: messages,
Signatures: signatures,
},
}, nil
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This specialization is unnecessary, right?

Comment on lines 361 to 362
messages = nil
signatures = nil
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You must also update signers in the same way as messages and signatures.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

Copy link
Contributor

@siburu siburu Feb 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dai1975 Not fixed yet? We need to reset signers by {batches[0].Signer, batches[1].Signer, ...} at the end of each iteration.

Daisuke Kanda added 2 commits February 9, 2026 09:57
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Signed-off-by: Daisuke Kanda <daisuke.kanda@datachain.jp>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants