Skip to content
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

Rename client_id field of the auth-register command to delegate_wallet_id #353

Open
devyetii opened this issue Dec 26, 2023 · 0 comments
Open

Comments

@devyetii
Copy link
Contributor

client_id field of the auth_register is used in the DelegateWallet field of the add_authorizer transaction, while client_key is used in the PublicKey field of the txn. They need to be not in correspondence with one another, since this will mean that the authorizer is trying to self-delegate (i.e. use the same wallet for operations and delegation), which is not allowed.
This field should be renamed to delegate_wallet_id to be clear that this will not correspond to the client_key.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant