Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix for panic. #4627

Merged
merged 1 commit into from
Feb 13, 2024
Merged

Fix for panic. #4627

merged 1 commit into from
Feb 13, 2024

Conversation

Frozen
Copy link
Contributor

@Frozen Frozen commented Feb 11, 2024

Fix for panic in mainnet.

@ONECasey
Copy link
Contributor

Waiting for testing to finish.

@ONECasey ONECasey assigned ONECasey and Frozen and unassigned ONECasey Feb 11, 2024
@ONECasey ONECasey added the high priority high priority issue with customer impact label Feb 11, 2024
@diego1q2w diego1q2w merged commit d5956ce into main Feb 13, 2024
4 checks passed
@diego1q2w diego1q2w deleted the fix/panic-mainnet branch February 13, 2024 08:21
ONECasey added a commit that referenced this pull request Feb 21, 2024
* fix allowed txs to be able to handle multiple txs for same from address (#4624)

* [HOTFIX] fix leader crosslink issue to not include old cross link in the propo… (#4629)

* fix leader crosslink issue to not include old cross link in the proposing block

* set higher epoch threshold for pending crosslinks to be added to proposing block

* delete old pending cross links

* delete when proposing

* delete when proposing

* delete when proposing

* delete when proposing

* minor logic change for the log

* minor logic change for the log

* minor logic change for the log

* minor logic change for the log

---------

Co-authored-by: Diego Nava <diego.nava77@hotmail.com>

* Fix for possible panic. (#4627)

* Fix, removed duplicated check.

---------

Co-authored-by: Gheis Mohammadi <Gheis.Mohammadi@gmail.com>
Co-authored-by: Diego Nava <diego.nava77@hotmail.com>
Co-authored-by: Konstantin <355847+Frozen@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
high priority high priority issue with customer impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants