Skip to content
This repository was archived by the owner on Jan 16, 2026. It is now read-only.

fix(peers): correct error message in OpStackEnr decode#3191

Merged
theochap merged 1 commit intoop-rs:mainfrom
zeroprooff:fix/enr-decode-error-message
Jan 8, 2026
Merged

fix(peers): correct error message in OpStackEnr decode#3191
theochap merged 1 commit intoop-rs:mainfrom
zeroprooff:fix/enr-decode-error-message

Conversation

@zeroprooff
Copy link
Contributor

Fixed a copy-paste error in the OpStackEnr decoder where the error message for version decoding incorrectly said "could not decode chain id" instead of "could not decode version". Small fix but makes debugging easier when ENR parsing fails.

@github-actions
Copy link
Contributor

github-actions bot commented Jan 3, 2026

This pull request has been automatically marked as stale because it has been inactive for 3 weeks.
It will be closed in 1 week if no further activity occurs.
If you believe this PR should remain open, please add the M-prevent-stale label or leave a comment.

@theochap theochap enabled auto-merge January 8, 2026 19:05
@codecov
Copy link

codecov bot commented Jan 8, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.8%. Comparing base (820ff00) to head (ae6bdc9).
⚠️ Report is 15 commits behind head on main.
✅ All tests successful. No failed tests found.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@theochap theochap added this pull request to the merge queue Jan 8, 2026
Merged via the queue into op-rs:main with commit 024aa7c Jan 8, 2026
45 of 47 checks passed
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 15, 2026
Fixed a copy-paste error in the OpStackEnr decoder where the error
message for version decoding incorrectly said "could not decode chain
id" instead of "could not decode version". Small fix but makes debugging
easier when ENR parsing fails.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants