Commitd7ba6c0

committed

fix IsGithubDotComURL check

1 parentd8d4b9b commitd7ba6c0Copy full SHA for d7ba6c0

File tree

-1

lines changed

-1

lines changed

Lines changed: 4 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -1096,7 +1096,10 @@ func (api API) userOAuth2Github(rw http.ResponseWriter, r http.Request) {`
`1096`	`1096`	`}`
`1097`	`1097`	`// If the user is logging in with github.com we update their associated`
`1098`	`1098`	`// GitHub user ID to the new one.`
`1099`		`-ifexternalauth.IsGithubDotComURL(api.GithubOAuth2Config.AuthCodeURL(""))&&user.GithubComUserID.Int64!=ghUser.GetID() {`
	`1099`	`+// We use AuthCodeURL from the OAuth2Config field instead of the one on`
	`1100`	`+// GithubOAuth2Config because when device flow is configured, AuthCodeURL`
	`1101`	`+// is overridden and returns a value that doesn't pass the URL check.`
	`1102`	`+ifexternalauth.IsGithubDotComURL(api.GithubOAuth2Config.OAuth2Config.AuthCodeURL(""))&&user.GithubComUserID.Int64!=ghUser.GetID() {`
`1100`	`1103`	`err=api.Database.UpdateUserGithubComUserID(ctx, database.UpdateUserGithubComUserIDParams{`
`1101`	`1104`	`ID:user.ID,`
`1102`	`1105`	`GithubComUserID: sql.NullInt64{`

Comments

(0)