Skip to content

Clean up lastRebroadcast on transaction rejection#2038

Merged
kamuikatsurgi merged 2 commits into0xPolygon:developfrom
LarryArnault45:txpool-rebroadcast-cleanup
Feb 26, 2026
Merged

Clean up lastRebroadcast on transaction rejection#2038
kamuikatsurgi merged 2 commits into0xPolygon:developfrom
LarryArnault45:txpool-rebroadcast-cleanup

Conversation

@LarryArnault45
Copy link
Copy Markdown
Contributor

Removes transaction hash from lastRebroadcast map when rejected during promotion, preventing memory accumulation.

@LarryArnault45 LarryArnault45 force-pushed the txpool-rebroadcast-cleanup branch 2 times, most recently from 1bdb40c to 72a78a3 Compare February 4, 2026 08:23
@cffls
Copy link
Copy Markdown
Contributor

cffls commented Feb 5, 2026

Thanks for the fix! Could you rebase on top of develop branch?

@kamuikatsurgi kamuikatsurgi force-pushed the txpool-rebroadcast-cleanup branch from 72a78a3 to e2bd018 Compare February 26, 2026 10:31
@sonarqubecloud
Copy link
Copy Markdown

@lucca30
Copy link
Copy Markdown
Contributor

lucca30 commented Feb 26, 2026

@claude Can you analyze the changes. Deep looking if there is any security breach introduced, check if the changes helps or somehow that can potentially lead to any bug?

@claude
Copy link
Copy Markdown

claude bot commented Feb 26, 2026

Claude encountered an error —— View job

Command failed: git fetch origin --depth=20 txpool-rebroadcast-cleanup

I'll analyze this and get back to you.

@kamuikatsurgi kamuikatsurgi merged commit fdbc857 into 0xPolygon:develop Feb 26, 2026
19 of 23 checks passed
lucca30 pushed a commit that referenced this pull request Mar 5, 2026
* Clean up lastRebroadcast on transaction rejection

* Add test for lastRebroadcast cleanup on promotion rejection

(cherry picked from commit fdbc857)
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.

4 participants