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

fix(cli): fix oauth flow #3169

Merged
merged 2 commits into from Sep 20, 2023
Merged

fix(cli): fix oauth flow #3169

merged 2 commits into from Sep 20, 2023

Conversation

jorgeepc
Copy link
Contributor

This PR fixes the cli oauth flow by adding a default path and fixing the Access-Control-Allow-Origin header.

Changes

  • add default path and
  • fix Access-Control-Allow-Origin header

Fixes

Checklist

  • tested locally
  • added new dependencies
  • updated the docs
  • added a test

@jorgeepc jorgeepc added the CLI label Sep 20, 2023
@jorgeepc jorgeepc self-assigned this Sep 20, 2023
@jorgeepc jorgeepc marked this pull request as ready for review September 20, 2023 16:26
@jorgeepc jorgeepc merged commit 23aff08 into main Sep 20, 2023
38 checks passed
@jorgeepc jorgeepc deleted the fix-cli-oauth-cors branch September 20, 2023 19:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@xoscar xoscar xoscar approved these changes

@danielbdias danielbdias danielbdias approved these changes

Assignees

@jorgeepc jorgeepc

Labels
CLI
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jorgeepc @xoscar @danielbdias