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

fix(auth): Updated vpc service version #328

Merged
merged 2 commits into from
Jun 30, 2022

Conversation

Aerex
Copy link
Collaborator

@Aerex Aerex commented Jun 30, 2022

Context

The PR will update the VPC service version to allow VPC authentication

Steps to Test

  1. Download CLI-SDK from this PR into local CLI: go get github.com/IBM-Cloud/ibm-cloud-cli-sdk@d8a884dadc543de917266dc6b67a3f7608bf94e3
  2. Build CLI
  3. Enable verbose logging: ibmcloud config --trace true
  4. Run the command ibmcloud login --vpc-cri
  5. Verify that the error invalid_version_range does not get returned

@Aerex Aerex requested a review from steveclay as a code owner June 30, 2022 16:23
@Aerex Aerex added do-not-merge Label to remind reviewers NOT merge this PR labels Jun 30, 2022
@steveclay steveclay removed do-not-merge Label to remind reviewers NOT merge this PR labels Jun 30, 2022
@@ -9,7 +9,7 @@ import (
)

const (
DefaultMetadataServiceVersion = "2021-09-30"
DefaultMetadataServiceVersion = "2022-06-30"
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please see the recommended change here #327 (comment)

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there an advantage to using the current date over the date in the defect link?

Copy link
Collaborator

@steveclay steveclay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems the version needs a precise date. Please see comment.

@Aerex Aerex requested a review from steveclay June 30, 2022 20:11
@steveclay steveclay merged commit f981a27 into dev Jun 30, 2022
@steveclay steveclay deleted the fix/update-default-metadata-vpc branch June 30, 2022 20:21
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 this pull request may close these issues.

2 participants