Skip to content

Oauth url auth params fix #180

Merged
aviadl merged 5 commits intomainfrom
oauth-url-auth-params-2
Apr 2, 2025
Merged

Oauth url auth params fix #180
aviadl merged 5 commits intomainfrom
oauth-url-auth-params-2

Conversation

@aviadl
Copy link
Copy Markdown
Member

@aviadl aviadl commented Apr 2, 2025

No description provided.

@aviadl aviadl requested review from gaokevin1 and slavikm April 2, 2025 16:02
slavikm
slavikm previously approved these changes Apr 2, 2025
View reviewed changes
@aviadl aviadl enabled auto-merge (squash) April 2, 2025 16:07
gaokevin1
gaokevin1 approved these changes Apr 2, 2025
View reviewed changes
Copy link
Copy Markdown
Member

@gaokevin1 gaokevin1 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

lgtm

The correct URL is being printed out.

result: https://oauth.google.com/auth?client_id=123&redirect_uri=http://localhost:8080/callback&access_type=offline&prompt=consent&flow=signup

@aviadl aviadl merged commit f8a5dc2 into main Apr 2, 2025
10 checks passed
@aviadl aviadl deleted the oauth-url-auth-params-2 branch April 2, 2025 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@gaokevin1 gaokevin1 gaokevin1 approved these changes

@slavikm slavikm slavikm left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@aviadl @slavikm @gaokevin1