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

feat(iota): don't require key generation for client.yaml if there is already one #3978

Merged
merged 5 commits into from
Nov 28, 2024

Conversation

Thoralf-M
Copy link
Member

@Thoralf-M Thoralf-M commented Nov 11, 2024

Description of change

Don't require key generation for client.yaml if there is already one. This could happen for example if iota keytool import was used before

Links to any relevant issues

Fixes #3972

Type of change

  • Enhancement (a non-breaking change which adds functionality)

How the change has been tested

Running cargo run client active-address with only existing iota.keystore and iota.aliases in the iota_config folder.

@Thoralf-M Thoralf-M added the dev-tools Issues related to the Developer Tools Team label Nov 11, 2024
@Thoralf-M Thoralf-M force-pushed the dev-tools/cli-existing-address branch from 8cd19c0 to f51d188 Compare November 11, 2024 12:00
@Thoralf-M Thoralf-M marked this pull request as ready for review November 11, 2024 12:18
@Thoralf-M Thoralf-M requested a review from a team as a code owner November 11, 2024 12:18
Copy link
Contributor

@DaughterOfMars DaughterOfMars left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that this fn is called prompt_if_no_config, are we sure this is possible?

@Thoralf-M
Copy link
Member Author

Given that this fn is called prompt_if_no_config, are we sure this is possible?

Yes, it happened to me with iota keytool import, prompt_if_no_config is not called there and should also not be called there as it doesn't require a client

@DaughterOfMars
Copy link
Contributor

Given that this fn is called prompt_if_no_config, are we sure this is possible?

Yes, it happened to me with iota keytool import, prompt_if_no_config is not called there and should also not be called there as it doesn't require a client

But isn't this change in that function?

@Thoralf-M
Copy link
Member Author

@DaughterOfMars I think I misunderstood you. The function is called prompt_if_no_config and it's checking inside if the config client.yaml exists, it doesn't check for iota.keystore.

@DaughterOfMars
Copy link
Contributor

@DaughterOfMars I think I misunderstood you. The function is called prompt_if_no_config and it's checking inside if the config client.yaml exists, it doesn't check for iota.keystore.

Oh okay 👍

Co-authored-by: Thibault Martinez <thibault@iota.org>
@thibault-martinez thibault-martinez merged commit 1dd90ac into develop Nov 28, 2024
38 of 39 checks passed
@thibault-martinez thibault-martinez deleted the dev-tools/cli-existing-address branch November 28, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev-tools Issues related to the Developer Tools Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Don't require keypair generation for client.yaml if there is already one
3 participants