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.

Sign up for GitHub

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

Jump to bottom

Breaking Change in User/Password/Token Order of Precedence #2109

Closed
Tracked by #2070
dkelosky opened this issue Apr 3, 2024 · 3 comments
Closed
Tracked by #2070

Breaking Change in User/Password/Token Order of Precedence #2109

dkelosky opened this issue Apr 3, 2024 · 3 comments
Assignees
@gejohnston
Labels
bug Something isn't working priority-high Production outage - this quarter or at least next quarter severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases

Comments

@dkelosky
Copy link
Contributor

dkelosky commented Apr 3, 2024 •
edited
Loading

Describe the bug
This commit is technically a breaking change for Zowe CLI.
zowe/imperative@262f4fc#diff-14634f70cce872732119d82fadcf7046800005fd18cca5ce7b7081672b9fdd2aL466

This changes the behavior in authentication order of precedence.

Expected and actual results
Can the change be investigated as to provide an option to revert to the old behavior?

Describe your environment

Additional context
@dkelosky dkelosky added bug Something isn't working new The issue wasn't triaged yet labels Apr 3, 2024
@github-project-automation github-project-automation bot moved this to New Issues in Zowe CLI Squad Apr 3, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 3, 2024

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.

@zFernand0
Copy link
Member

We will implement as an optional constructor-level parameter. Said parameter will default to the original behavior in V2, and we will flip the value to the new behavior on V3.

@zFernand0 zFernand0 added priority-high Production outage - this quarter or at least next quarter severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases and removed new The issue wasn't triaged yet labels Apr 8, 2024
@zowe-robot zowe-robot moved this from New Issues to High Priority in Zowe CLI Squad Apr 8, 2024
@zFernand0 zFernand0 self-assigned this Apr 8, 2024
@adam-wolfe adam-wolfe mentioned this issue Apr 10, 2024
Closed
23 tasks
@gejohnston gejohnston self-assigned this Apr 11, 2024
@gejohnston gejohnston mentioned this issue Apr 15, 2024
Merged
4 tasks
@JTonda JTonda moved this from High Priority to In Progress in Zowe CLI Squad Apr 15, 2024
@zFernand0 zFernand0 removed their assignment Apr 22, 2024
@gejohnston
Copy link
Member

Fixed as part of the pull request "Reinstate token precedence over password in AbstractRestClient" #2119

@github-project-automation github-project-automation bot moved this from In Progress to Closed in Zowe CLI Squad Apr 30, 2024
@gejohnston gejohnston mentioned this issue May 1, 2024
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gejohnston gejohnston

Labels
bug Something isn't working priority-high Production outage - this quarter or at least next quarter severity-high Bug for which there may be workaround but limits the usage of the Zowe for major use cases
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dkelosky @gejohnston @zFernand0