Merge pull request #365 from JokerQyou/patch-1

lakhansamani · web-flow · commit 9ae616b6b547 · 2023-06-30T18:10:31.000+05:30
Fix wrong response_type parsed when missing response_mode
diff --git a/server/handlers/authorize.go b/server/handlers/authorize.go
@@ -84,9 +84,9 @@ func AuthorizeHandler() gin.HandlerFunc {
 
 		if responseMode == "" {
 			if val, err := memorystore.Provider.GetStringStoreEnvVariable(constants.EnvKeyDefaultAuthorizeResponseMode); err == nil {
-				responseType = val
+				responseMode = val
 			} else {
-				responseType = constants.ResponseModeQuery
+				responseMode = constants.ResponseModeQuery
 			}
 		}
 

Original file line number	Diff line number	Diff line change
`@@ -84,9 +84,9 @@ func AuthorizeHandler() gin.HandlerFunc {`
`84`	`84`
`85`	`85`	`if responseMode == "" {`
`86`	`86`	`if val, err := memorystore.Provider.GetStringStoreEnvVariable(constants.EnvKeyDefaultAuthorizeResponseMode); err == nil {`
`87`		`- responseType = val`
	`87`	`+ responseMode = val`
`88`	`88`	`} else {`
`89`		`- responseType = constants.ResponseModeQuery`
	`89`	`+ responseMode = constants.ResponseModeQuery`
`90`	`90`	`}`
`91`	`91`	`}`
`92`	`92`