Skip to content

Commit

Permalink
Merge pull request #13 from supabase/feature/query-param-callback
Browse files Browse the repository at this point in the history
fix: changes # to ? in external redirect callbacks
  • Loading branch information
awalias authored Sep 22, 2020
2 parents ac0ea02 + 9ec992d commit ccbec67
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions api/external.go
Original file line number Diff line number Diff line change
Expand Up @@ -212,7 +212,7 @@ func (a *API) internalExternalProviderCallback(w http.ResponseWriter, r *http.Re
q.Set("token_type", token.TokenType)
q.Set("expires_in", strconv.Itoa(token.ExpiresIn))
q.Set("refresh_token", token.RefreshToken)
rurl += "#" + q.Encode()
rurl += "?" + q.Encode()
}
http.Redirect(w, r, rurl, http.StatusFound)
return nil
Expand Down Expand Up @@ -322,7 +322,7 @@ func (a *API) redirectErrors(handler apiHandler, w http.ResponseWriter, r *http.
err := handler(w, r)
if err != nil {
q := getErrorQueryString(err, errorID, log)
http.Redirect(w, r, a.getExternalRedirectURL(r)+"#"+q.Encode(), http.StatusFound)
http.Redirect(w, r, a.getExternalRedirectURL(r)+"?"+q.Encode(), http.StatusFound)
}
}

Expand Down

0 comments on commit ccbec67

Please sign in to comment.