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

fix(node): use canonical --l2-engine-jwt-secret in CLI example#2822

Merged
emhane merged 1 commit intoop-rs:mainfrom
MozirDmitriy:chore/fix-node-cli-jwt-flag-example
Sep 9, 2025
Merged

fix(node): use canonical --l2-engine-jwt-secret in CLI example#2822
emhane merged 1 commit intoop-rs:mainfrom
MozirDmitriy:chore/fix-node-cli-jwt-flag-example

Conversation

@MozirDmitriy
Copy link
Contributor

Replace incorrect --l2-jwt-secret in the inline example with the canonical --l2-engine-jwt-secret. This aligns the comment with the actual Clap flag (l2_engine_jwt_secret) and the CLI reference/README/docker examples, preventing confusion for users copying the example.

@codecov
Copy link

codecov bot commented Sep 8, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.0%. Comparing base (a4a6101) to head (3716df8).
⚠️ Report is 6 commits behind head on main.

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

@emhane emhane merged commit 8323114 into op-rs:main Sep 9, 2025
35 of 36 checks passed
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Dec 10, 2025
…/kona#2822)

Replace incorrect --l2-jwt-secret in the inline example with the
canonical --l2-engine-jwt-secret. This aligns the comment with the
actual Clap flag (l2_engine_jwt_secret) and the CLI
reference/README/docker examples, preventing confusion for users copying
the example.
theochap pushed a commit to ethereum-optimism/optimism that referenced this pull request Jan 14, 2026
…/kona#2822)

Replace incorrect --l2-jwt-secret in the inline example with the
canonical --l2-engine-jwt-secret. This aligns the comment with the
actual Clap flag (l2_engine_jwt_secret) and the CLI
reference/README/docker examples, preventing confusion for users copying
the example.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants