use JWT when and only when the user specifies a JWT secret explicitly #3786
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This has some advantages:
It allows people to configure and test with JWT without being on a merge testnet.
When Prater and mainnet do each get a non-
FAR_FUTURE_EPOCH
BELLATRIX_FORK_EPOCH
but before the merge actually happens, it won't break all those setups using those incompatible Web3 providers; rather, it'll be user choice/action determining this.it de-emphasizes the less useful case of using its CL-autogenerated JWT secret, because generally one's would launch the EL first, and it will generate a secret. For an EL, this is a reasonable approach. For a CL, it seldom makes sense to any default generated JWT secret.
Because of the last point, even in the testnets, there will be virtually no breakage from changing the edge case-semantics of this option.