-
Notifications
You must be signed in to change notification settings - Fork 92
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
Allow secure credentials to be enabled in Theia #2218
Conversation
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
You have successfully added a new SonarCloud configuration ``. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab. |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #2218 +/- ##
==========================================
- Coverage 91.53% 91.53% -0.01%
==========================================
Files 89 89
Lines 8864 8880 +16
Branches 1829 1837 +8
==========================================
+ Hits 8114 8128 +14
- Misses 749 751 +2
Partials 1 1
... and 1 file with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report in Codecov by Sentry. |
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
… Theia Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
After further testing, it seems like the "Secure Credentials Enabled" setting is always enforced when v2 config with "secure" arrays is present. This behavior is the same whether in VS Code or in Theia. I think this PR is still a good change, to avoid forcing the setting to always be false. If we want to disallow loading secure credentials when "Secure Credentials Enabled" is false, that would require code changes in Imperative and I would suggest that it be a separate effort. |
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! thanks @t1m0thyj for the fix
Signed-off-by: Timothy Johnson <timothy.johnson@broadcom.com>
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Proposed changes
Fixed secure credential storage not possible to enable in Theia. It remains disabled by default in Theia where support for secure storage is limited.
Also improved error handling for the
ProfilesUtils.initializeZoweProfiles
method. This is a follow up to #2188 to include some fixes that were missed. Moved some of theProfilesUtils
methods into a static class to make them easier to test.Release Notes
Milestone: 2.8.0
Changelog: Fixed secure credential storage not possible to enable in Theia.
Types of changes
What types of changes does your code introduce to Zowe Explorer?
Put an
x
in the boxes that applyChecklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This checklist will be used as reference for both the contributor and the revieweryarn workspace vscode-extension-for-zowe vscode:prepublish
has been executedFurther comments