Make the nickname claim conditional on the openid scope too

This is only needed for OIDC anyway
2023-02-24 16:27:01 +01:00 · 2023-02-24 16:27:01 +01:00 · aec65d8e19
parent f4be6e135a
commit aec65d8e19
1 changed files with 4 additions and 3 deletions
--- a/handlers/user/info.go
+++ b/handlers/user/info.go
@ -61,9 +61,10 @@ func UserInfo(ctx *gin.Context) {

 			if s == "openid" {
 				userinfoRes["sub"] = *tokenInfo.Sub
-			} else if s == "username" {
-				// Add "nickname" field for OIDC compatibility
-				userinfoRes["nickname"] = userinfoRes[s]
+				if value, ok := (*userinfo)["username"]; ok {
+					// Add "nickname" field for OIDC compatibility
+					userinfoRes["nickname"] = value
+				}
 			}
 		}
 	}