Merge pull request #497 from ev1lQuark/fix-PKCE

techknowlogick · web-flow · commit aeec8c8c851e · 2023-02-26T14:09:39.000-05:00
fix: fitbit PKCE param
diff --git a/providers/fitbit/session.go b/providers/fitbit/session.go
@@ -31,7 +31,7 @@ func (s Session) GetAuthURL() (string, error) {
 // token to be stored for future use.
 func (s *Session) Authorize(provider goth.Provider, params goth.Params) (string, error) {
 	p := provider.(*Provider)
-	token, err := p.config.Exchange(oauth2.NoContext, params.Get("code"))
+	token, err := p.config.Exchange(oauth2.NoContext, params.Get("code"), oauth2.SetAuthURLParam("code_verifier", params.Get("code_verifier")))
 	if err != nil {
 		return "", err
 	}

Original file line number	Diff line number	Diff line change
`@@ -31,7 +31,7 @@ func (s Session) GetAuthURL() (string, error) {`
`31`	`31`	`// token to be stored for future use.`
`32`	`32`	`func (s *Session) Authorize(provider goth.Provider, params goth.Params) (string, error) {`
`33`	`33`	`p := provider.(*Provider)`
`34`		`- token, err := p.config.Exchange(oauth2.NoContext, params.Get("code"))`
	`34`	`+ token, err := p.config.Exchange(oauth2.NoContext, params.Get("code"), oauth2.SetAuthURLParam("code_verifier", params.Get("code_verifier")))`
`35`	`35`	`if err != nil {`
`36`	`36`	`return "", err`
`37`	`37`	`}`