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

ProfileInfo.updateKnownProperty rewrites zowe.config.json file when updating a secure value #2324

Closed
t1m0thyj opened this issue Oct 28, 2024 · 1 comment · Fixed by #2323 or #2313
Closed

ProfileInfo.updateKnownProperty rewrites zowe.config.json file when updating a secure value #2324

t1m0thyj opened this issue Oct 28, 2024 · 1 comment · Fixed by #2323 or #2313
Assignees
@t1m0thyj
Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-low Bug that makes the usage of the Zowe less convenient but doesn't impact key use cases

Comments

@t1m0thyj
Copy link
Member

When Zowe Explorer updates a user or password in the secure vault, it also rewrites the zowe.config.json file which triggers a file watcher event.

Perhaps the Imperative ProfileInfo API could check if a property is already defined as a secure value in the team config file before rewriting it to disk.
@t1m0thyj t1m0thyj added bug Something isn't working new The issue wasn't triaged yet labels Oct 28, 2024
@github-project-automation github-project-automation bot moved this to New Issues in Zowe CLI Squad Oct 28, 2024
@t1m0thyj t1m0thyj self-assigned this Oct 28, 2024
@github-actions GitHub Actions
Copy link

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.

@t1m0thyj t1m0thyj linked a pull request Oct 28, 2024 that will close this issue
fix: Ignore events within the app that triggered them #2323
Merged
4 tasks
@JTonda JTonda added priority-medium Not functioning - next quarter if capacity permits severity-low Bug that makes the usage of the Zowe less convenient but doesn't impact key use cases and removed new The issue wasn't triaged yet labels Nov 4, 2024
@zowe-robot zowe-robot moved this from New Issues to Medium Priority in Zowe CLI Squad Nov 4, 2024
@JTonda JTonda moved this from Medium Priority to In Progress in Zowe CLI Squad Nov 4, 2024
@t1m0thyj t1m0thyj mentioned this issue Nov 5, 2024
Merged
4 tasks
@t1m0thyj t1m0thyj linked a pull request Nov 11, 2024 that will close this issue
Fix Config API bugs that affect extenders converting V1 profiles #2313
Merged
4 tasks
@github-project-automation github-project-automation bot moved this from In Progress to Closed in Zowe CLI Squad Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@t1m0thyj t1m0thyj

Labels
bug Something isn't working priority-medium Not functioning - next quarter if capacity permits severity-low Bug that makes the usage of the Zowe less convenient but doesn't impact key use cases
Projects
Zowe CLI Squad
Status: Closed
Milestone
No milestone
2 participants
@t1m0thyj @JTonda