-
Notifications
You must be signed in to change notification settings - Fork 21
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
inconsistent representation of commitment ATX and node ID #163
Comments
The node ID is stored in hex in key.bin and base64 in postdata_metadata.json. This is really confusing and frustrating. |
key.bin contains the private key hex-encoded. |
Right. IMHO they should both be in hex :) (The private key includes the public key so at least this way you can do pattern-matching.) |
I also responded in this comment: #203 (comment) Updating this is more involved than it initially appears. We need a migration plan such that existing PoSTs are updated from the current schema to the new one. This is btw. true for any file layout we want to change and is probably worth documenting a general approach for as a cross cutting concern 🙂 |
We already had that issue come up in that some PoSTs have a Thankfully there was an easy migration path that allowed to just add the value if not already present. |
Problem statement
Node ID and commitment ATX ID are formatted in base64 both in postdata_metadata.json and in post-rs CLI initializer. However, postcli accepts them in hex. They are also formatted in hex when logged in go-spacemesh. It is inconsistent and causes extra work to convert.
DoD
We should format ATXes in the same way everywhere (either hex or base64 - this is to decide).
The text was updated successfully, but these errors were encountered: