Skip to content

Fix issue #2070. Don't fail context update when there is a record of rejected transaction - #2098

Merged
LGLO merged 3 commits into
mainfrom
fix-issue-2070
Sep 11, 2026
Merged

Fix issue #2070. Don't fail context update when there is a record of rejected transaction#2098
LGLO merged 3 commits into
mainfrom
fix-issue-2070

Conversation

@LGLO

@LGLO LGLO commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

Overview

Fixed #2070 by not crashing replay when there is a send_mn_transaction record of a transaction that wasn't included in Midnight Ledger because it was not well formed.

🗹 TODO before merging

  • Ready

📌 Submission Checklist

  • All commits are signed off (git commit -s) for the DCO
  • Changes are backward-compatible (or flagged if breaking)
  • Pull request description explains why the change is needed
  • Self-reviewed the diff
  • I have included a change file, or skipped for this reason:
  • If the changes introduce a new feature, I have bumped the node minor version
  • Update documentation (if relevant)
  • Updated AGENTS.md if build commands, architecture, or workflows changed
  • No new todos introduced

🧪 Testing Evidence

Please describe any additional testing aside from CI:

  • Additional tests are provided (if possible)

🔱 Fork Strategy

  • Node Runtime Update
  • Node Client Update
  • Other:
  • N/A

Links

@LGLO
LGLO requested a review from a team as a code owner September 4, 2026 15:46

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 263d604ec6

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ledger/helpers/src/ledger_8/context.rs Outdated
Comment thread ledger/helpers/src/ledger_8/context.rs Outdated
Comment thread ledger/helpers/unsafe/src/ledger_8/context.rs
send_mn_transaction record of a transaction that wasn't included in Midnight Ledger.

Signed-off-by: Lech Głowiak <lech.glowiak@shielded.io>
@LGLO
LGLO requested a review from gilescope September 10, 2026 09:17
@LGLO LGLO changed the title Fix issue #2070. Don't fail context update when there is a Fix issue #2070. Don't fail context update when there is a record of rejected transaction Sep 11, 2026

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 8daa5ea831

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ledger/helpers/unsafe/src/ledger_9/context.rs
Signed-off-by: Lech Głowiak <lech.glowiak@shielded.io>

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9a76b477b6

ℹ️ About Codex in GitHub

Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".

Comment thread ledger/helpers/unsafe/src/ledger_8/context.rs
@LGLO
LGLO added this pull request to the merge queue Sep 11, 2026
Merged via the queue into main with commit 181a07c Sep 11, 2026
41 checks passed
@LGLO
LGLO deleted the fix-issue-2070 branch September 11, 2026 15:47
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.

toolkit: cannot sync against Preview — replay aborts on a chain-accepted transaction

2 participants