Skip to content

fix: don't crash MPC node on startup for failed attestation submission - #1772

Merged
DSharifi merged 3 commits into
mainfrom
1763-bug-mpc-node-crashes-if-attestation-submission-happens-before-the-node-keys-are-added-to-the-near-account
Jan 16, 2026
Merged

DSharifi merged 3 commits into
mainfrom
1763-bug-mpc-node-crashes-if-attestation-submission-happens-before-the-node-keys-are-added-to-the-near-account

Conversation

@DSharifi

Copy link
Copy Markdown
Contributor

closes #1763

It's fine that we remove this manual attestation submission, since we spawn the periodic attestation submission just below.

Since we currently don't detach the submission in a separate task, the node will wait with starting the coordinator until the submission succeeds. If it fails, the node will crash itself which is not desirable.

netrome
netrome previously approved these changes Jan 16, 2026

@netrome netrome left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks!

@gilcu3 gilcu3 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Nice!

@DSharifi
DSharifi enabled auto-merge January 16, 2026 13:05
…mission-happens-before-the-node-keys-are-added-to-the-near-account
@DSharifi
DSharifi disabled auto-merge January 16, 2026 14:09
@DSharifi
DSharifi enabled auto-merge January 16, 2026 14:09
@DSharifi
DSharifi disabled auto-merge January 16, 2026 14:37
@DSharifi
DSharifi added this pull request to the merge queue Jan 16, 2026
Merged via the queue into main with commit 87122b6 Jan 16, 2026
7 checks passed
@DSharifi
DSharifi deleted the 1763-bug-mpc-node-crashes-if-attestation-submission-happens-before-the-node-keys-are-added-to-the-near-account branch January 16, 2026 15:00
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.

Bug: MPC node crashes if attestation submission happens before the node keys are added to the near account

3 participants