Supabase
Bug: QueryParams is not working #20439
Replies: 6 comments
-
|
Related discussion: |
Beta Was this translation helpful? Give feedback.
-
|
is the expected behavior here that the If so, it looks like this is the culprit. |
Beta Was this translation helpful? Give feedback.
-
|
For me, yeah I was expecting it to be appended to the redirect to URL. |
Beta Was this translation helpful? Give feedback.
-
|
The docs are not super clear here:
I think we need a member of the team to chime in on the expected behavior here. If we are expecting If we are expecting |
Beta Was this translation helpful? Give feedback.
-
|
And it would be great to have some idea about appending the query Params in the redirectTo if it was to be consumed with the OAuth provider.
We solved this manually before, by appending a string to the redirect to URL
Sent from mobile device
…________________________________
From: marcus dylan johnson ***@***.***>
Sent: Monday, July 3, 2023 4:19:37 PM
To: supabase/supabase ***@***.***>
Cc: Abdurrahman Rajab ***@***.***>; Author ***@***.***>
Subject: Re: [supabase/supabase] Bug: QueryParams is not working (Issue #14695)
The docs<https://supabase.com/docs/reference/javascript/auth-signinwithsso> are not super clear here:
queryParams An object of query params
I think we need a member of the team to chime in on the expected behavior here.
If we are expecting queryParams to be appended to the whole URL (I'm assuming to be consumed by the OAuth provider), then we should probably specify that.
If we are expecting queryParams to be appended to the redirectTo URL, then I think we need to make some changes to the code linked above.
—
Reply to this email directly, view it on GitHub<#14695 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/AELNMKOP6XX2UEEP3TFNU5TXOLBGTANCNFSM6AAAAAAYUMRZEA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
Beta Was this translation helpful? Give feedback.
-
|
Is this dead? It seems to be a bug to me... |
Beta Was this translation helpful? Give feedback.
-
Bug report
Describe the bug
In this PR: open-sauced/app#1184
We are using auto-signin from SupaBase with redirect and queryParams option, I am expecting the QueryParams to be parsed and passed with react but it's not.
Not sure, if this is a bug or the QueryParams option is expected to do something different.
To Reproduce
Steps to reproduce the behavior, please provide code snippets or a repository:
Expected behavior
Eexpected to be able to get the query params in the redirected URL.
Screenshots
If applicable, add screenshots to help explain your problem.
System information
Additional context
Add any other context about the problem here.
Beta Was this translation helpful? Give feedback.
All reactions