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

Update Zowe VSCode extension #1

Closed
6 tasks done
JTonda opened this issue Oct 9, 2018 · 3 comments · Fixed by #14
Closed
6 tasks done

Update Zowe VSCode extension #1

JTonda opened this issue Oct 9, 2018 · 3 comments · Fixed by #14
Assignees

Comments

@JTonda
Copy link

JTonda commented Oct 9, 2018

As Michelle, I want the Zowe VSCode Extension to work properly with profiles stored in plain text so that I can continue to make use of it to improve my interaction with z/OS datasets.

Update Zowe VSCode extension to work with profile credentials now stored in plain text under .zowe. Update documentation. No requirement to also work with securely stored profiles (eg 1.x).

Enhance error message to say that version is incompatible, please upgrade to the latest version. See comment below.

Depends on zowe/zowe-cli#7
'Replace Keytar - Store creds in plain text' in Zowe CLI repo

  • Bump versions of zowe and imperative
    Status: Completed
    Assigned: @AHumanFromCA

  • Remove calls to CredentialManager and keytar
    Status: Completed
    Assigned: @AHumanFromCA

  • Initialize logger to prevent possible problems
    Status: Completed
    Assigned: @lyzla01

  • Modify CliProfileManager calls to account for plaintext credentials
    Status: It just worked without any changes :)
    Assigned:

  • Run system tests
    Status: Completed
    Assigned: @AHumanFromCA

  • Update documentation
    Status:
    Assigned: @JamesBauman

*Accept feature with PM when complete (This & zowe/zowe-cli#16 both in this Sprint)
8 point story

@JTonda JTonda added this to the CA - 18PI4S3 milestone Oct 9, 2018
@JTonda JTonda assigned ghost and JamesBauman Oct 9, 2018
@ghost
Copy link

ghost commented Oct 9, 2018

Meeting Notes

  • Remove calls to CredentialManager / keytar
  • Initialize the logger
  • Possible updates to CliProfileManager call to account for no credential manager
  • Update documentation
    • Remove doc about keytar
  • Publish via pipeline not part of this story
  • Bump version of zowe and imperative
  • Run existing system tests. They should all pass

@ghost
Copy link

ghost commented Oct 9, 2018

After further discussions this ask might be impossible:

Enhance error message to say that version is incompatible, please upgrade to the latest version.

Reasons

  • The meta file for profiles does not contain any versioning information
  • The new system can work without zowe-cli being installed as all profiles can be hand created

Notes:

  • As is currently, if there are missing properties, the CliProfileManager will inform of the missing fields
Command Error:
Expect Error: Required parameter 'hostname' must be defined

@ghost ghost added in progress and removed backlog labels Oct 9, 2018
ghost pushed a commit that referenced this issue Oct 10, 2018
@ghost
Copy link

ghost commented Oct 11, 2018

At this point, the story wasn't to enable plaintext profiles. I think the actual story should be more like, bring VSC Extension to production quality :/

@ghost ghost added review and removed in progress labels Oct 15, 2018
@ghost ghost removed the review label Oct 15, 2018
kristinochka added a commit to kristinochka/vscode-extension-for-zowe that referenced this issue Apr 22, 2019
phaumer pushed a commit that referenced this issue Apr 22, 2020
bring in changes from head repo
@crawr crawr mentioned this issue Feb 16, 2021
6 tasks
lauren-li pushed a commit that referenced this issue Jun 8, 2021
Close connection - update with master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants