-
Notifications
You must be signed in to change notification settings - Fork 11.1k
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
[cli/alias] Keytool update alias #15162
Merged
stefan-mysten
merged 9 commits into
MystenLabs:main
from
stefan-mysten:keytool_update_alias
Dec 11, 2023
Merged
[cli/alias] Keytool update alias #15162
stefan-mysten
merged 9 commits into
MystenLabs:main
from
stefan-mysten:keytool_update_alias
Dec 11, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The latest updates on your projects. Learn more about Vercel for Git ↗︎
3 Ignored Deployments
|
9212fd2
to
acd1902
Compare
dariorussi
approved these changes
Dec 8, 2023
b04c190
to
d1b6e3c
Compare
This reverts commit d1b6e3c.
821dfab
to
c0e0b25
Compare
gdanezis
pushed a commit
that referenced
this pull request
Dec 15, 2023
## Description Provide a new command for updating the alias of an address. The new command `keytool update-alias` takes an old alias that needs to be updated, and optionally, a new alias name. If there is no new alias given, a new random alias will be generated. Furthermore, the alias must start with a letter and can contain only letters, digits, hyphens (-), or underscores (_). An example for using this command is: `sui keytool update-alias test high_yield_address`, which outputs the following text: `Old alias test was updated to high_yield_address` ## Test Plan Two new tests for FileBasedKeystore and InMemKeystore. <img width="869" alt="image" src="https://github.com/MystenLabs/sui/assets/135084671/2f3cecd1-d283-4a9d-8995-d3999888a9e9"> --- If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process. ### Type of Change (Check all that apply) - [ ] protocol change - [x] user-visible impact - [ ] breaking change for a client SDKs - [ ] breaking change for FNs (FN binary must upgrade) - [ ] breaking change for validators or node operators (must upgrade binaries) - [ ] breaking change for on-chain data layout - [ ] necessitate either a data wipe or data migration ### Release notes Added support for updating an alias of an address in the Sui CLI. The alias must start with a letter and can contain only letters, digits, hyphens (-), or underscores (_). Try it out through this command: `sui keytool update-alias --help`
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Description
Provide a new command for updating the alias of an address. The new command
keytool alias
has one flag--update
, followed by an old alias that needs to be updated, and optionally, a new alias name. If there is no new alias given, a new random alias will be generated.An example for using this command is:
sui keytool update-alias test high_yield_address
, which outputs the following text:Old alias test was updated to high_yield_address
Test Plan
Two new tests for FileBasedKeystore and InMemKeystore.
![image](https://private-user-images.githubusercontent.com/135084671/288617368-2f3cecd1-d283-4a9d-8995-d3999888a9e9.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjEzNTc3MjEsIm5iZiI6MTcyMTM1NzQyMSwicGF0aCI6Ii8xMzUwODQ2NzEvMjg4NjE3MzY4LTJmM2NlY2QxLWQyODMtNGE5ZC04OTk1LWQzOTk5ODg4YTllOS5wbmc_WC1BbXotQWxnb3JpdGhtPUFXUzQtSE1BQy1TSEEyNTYmWC1BbXotQ3JlZGVudGlhbD1BS0lBVkNPRFlMU0E1M1BRSzRaQSUyRjIwMjQwNzE5JTJGdXMtZWFzdC0xJTJGczMlMkZhd3M0X3JlcXVlc3QmWC1BbXotRGF0ZT0yMDI0MDcxOVQwMjUwMjFaJlgtQW16LUV4cGlyZXM9MzAwJlgtQW16LVNpZ25hdHVyZT02MWY0ZDkyYTQ5MDE2ZDFlNDE0ZTlkZDA2MzE1M2JkYmU1ZTIwMWVhYmE5ZmYzMGM1NjI5YTRkNWM1YjY0MGRmJlgtQW16LVNpZ25lZEhlYWRlcnM9aG9zdCZhY3Rvcl9pZD0wJmtleV9pZD0wJnJlcG9faWQ9MCJ9.6Vcmcb7wQ_M4YmU6OCKcX9BL56neEgYsxalrIY0E7X8)
If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)
Release notes
Added support for updating an alias of an address in the Sui CLI. See the console help for this command using:
sui keytool update-alias --help
.