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

Config auto-init behaves strangely with empty config #1862

Open
t1m0thyj opened this issue Aug 1, 2023 · 1 comment
Open

Config auto-init behaves strangely with empty config #1862

t1m0thyj opened this issue Aug 1, 2023 · 1 comment
Labels
bug Something isn't working priority-low Legit issue but cosmetic or nice-to-have severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.

Comments

@t1m0thyj
Copy link
Member

t1m0thyj commented Aug 1, 2023

Steps to Reproduce

  1. Create an empty config file: zowe config init --no-prompt
  2. Run auto-init specifying host and port: zowe config auto-init --host example.com --port 7554
  3. Enter username and password when prompted

Actual Behavior

  • User, password, and token value are all stored by the credential manager. User and password get stored because autoStore is true.
  • The hostname specified on the command line is used to obtain the token but does not get stored in the config file.

Expected Behavior

  • User and password should not be stored because they would take precedence over the token value. This could be fixed by passing autoStore: false to the prompting method like we do in BaseAuthHandler.
  • The hostname specified on the command line should replace the empty hostname in the config file.
@t1m0thyj t1m0thyj added the bug Something isn't working label Aug 1, 2023
@github-actions
Copy link

github-actions bot commented Aug 1, 2023

Thank you for creating a bug report.
We will investigate the bug and evaluate its impact on the product.
If you haven't already, please ensure you have provided steps to reproduce the bug and as much context as possible.

@JTonda JTonda added priority-low Legit issue but cosmetic or nice-to-have severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex. labels Aug 16, 2023
@awharn awharn transferred this issue from zowe/imperative Nov 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working priority-low Legit issue but cosmetic or nice-to-have severity-medium Bug where workaround exists or that doesn't prevent the usage of Zowe. Just makes it more complex.
Projects
Status: Low Priority
Development

No branches or pull requests

2 participants