-
Notifications
You must be signed in to change notification settings - Fork 619
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
Revert "json: remove deprecated old style path+data request format" #7797
Conversation
@posvyatokum, we’ll need another rc with this cherry picked. |
/// Deprecated. If you want to decode a CryptoHash, use CryptoHash::from_str. | ||
/// For anything else, don’t use base58. This is still here because of | ||
/// deprecated RPC query format in RpcQueryRequest::parse. | ||
pub fn from_base58(s: &str) -> Result<Vec<u8>, Box<dyn std::error::Error + Send + Sync>> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In a follow-up, lets move this closed to the use-site, so that we don't have to mark it as deprecated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Counter argument though is that it would add direct bs58
dependency in chain/jsonrpc
which we’ll forget to remove once from_base58
is removed.
@mina86 Testnet will switch to new protocol version in the next epoch. If everything goes well, I imagine doing a release a day after that, so on Wednesday. If something breaks with protocol upgrade, the release date will be determined by that bugfix. Is that timeline ok or is this more urgent? |
Sure. Most importantly is that this revert gets into mainnet release. |
Turns out there still are users of the old query format. Bring it back.
This reverts commit be8a1ea.
Fixes: #7769