-
Notifications
You must be signed in to change notification settings - Fork 118
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
[ARC-81] Partkey integrity hash #327
base: main
Are you sure you want to change the base?
[ARC-81] Partkey integrity hash #327
Conversation
3605662
to
be7738d
Compare
Given that participation keys are not portable between networks for the majority of users, should we include the network in the integrity hash? (e.g. genesis hash) This would prevent network mismatch accidents, where e.g. a keyreg meant for testnet ended up on mainnet. |
The ARC-78 URL strives to be as minimal as possible and does not include GH or network-id . |
Not sure how it makes them incompatible. If ARC78 QR doesn't encode network/GH, that is all the more reason to include it in the integrity hash. The information (GH) would be independently known everywhere where you would want to generate an integrity hash - node, wallets, explorers.
Arguably ARC78 should have included network as well - in the TUI we currently have to warn users to make sure they are on the correct network when going through the QR. But even if we don't, not sure what would be stopping the wallet from including its current network GH in the integrity hash calculation * ARC-78 is status final |
Second thread: Should this cover keyreg offline txns as well as @urtho suggested offline? If so, what do you think about same payload structure, including the keys & rounds as all-zero? |
Good questions, the standard way is to hash empty values - that is zero filled arrays instead of skipping them. |
No description provided.