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

chore: explicitly set token authentication when using the CLI #509

Closed
wants to merge 1 commit into from

Conversation

PeterSchafer
Copy link
Contributor

@PeterSchafer PeterSchafer commented May 15, 2024

Description

The CLI default authentication will be switching to OAuth, since snyk-ls uses the CLI for token based authentication, it is necessary to explicitly state the auth-type to be token.

This can be tested with the preview version of the CLI, which already uses OAuth as default.
The expected behaviour is that snyk-ls still uses token based authentication unless it is connecting to fedramp instances or the authentication type is changed.

Checklist

  • Tests added and all succeed
  • Linted
  • README.md updated, if user-facing
  • License file updated, if new 3rd-party dependency is introduced

🚨After having merged, please update the CLI go.mod to pull in latest language server.

Screenshots / GIFs

Visuals that may help the reviewer. Please add screenshots for any UI change. GIFs are most welcome!

@PeterSchafer PeterSchafer force-pushed the chore/CLI-326_support_oauth_default branch from 872b87e to 2176270 Compare May 15, 2024 16:40
@PeterSchafer PeterSchafer marked this pull request as ready for review May 15, 2024 16:40
@PeterSchafer PeterSchafer requested a review from a team as a code owner May 15, 2024 16:40
@PeterSchafer PeterSchafer marked this pull request as draft May 15, 2024 18:16
Copy link
Contributor

@cmars cmars left a comment

Choose a reason for hiding this comment

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

Tested locally with VSCode.

Wondering why oauth didn't work... maybe because we're testing the token v1-style instead of using /rest/self in LS?

@PeterSchafer
Copy link
Contributor Author

Closing since this change will be replaced by another solution in the CLI

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