Skip to content

Commit 669d946

Browse files
committed
Flip global name and display name
1 parent 60ca215 commit 669d946

File tree

4 files changed

+7
-7
lines changed

4 files changed

+7
-7
lines changed

API/Controller/OAuth/SignupGetData.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -59,7 +59,7 @@ public async Task<IActionResult> OAuthSignupGetData([FromRoute] string provider)
5959
{
6060
Provider = auth.Provider,
6161
Email = auth.Principal.FindFirst(ClaimTypes.Email)?.Value,
62-
DisplayName = auth.Principal.FindFirst(ClaimTypes.Name)?.Value ?? auth.Principal.FindFirst(OAuthConstants.ClaimGlobalName)?.Value,
62+
DisplayName = auth.ExternalAccountDisplayName ?? auth.ExternalAccountName,
6363
ExpiresAt = auth.Properties.ExpiresUtc!.Value.UtcDateTime
6464
});
6565
}

API/Controller/OAuth/_ApiController.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -76,15 +76,15 @@ private async Task<OneOf<ValidatedFlowContext, OAuthValidationError>> ValidateOA
7676
return OAuthValidationError.FlowDataMissingOrInvalid;
7777
}
7878

79-
string? displayName = auth.Principal.FindFirst(ClaimTypes.Name)?.Value;
80-
string? globalName = auth.Principal.FindFirst(OAuthConstants.ClaimGlobalName)?.Value;
79+
var name = auth.Principal.FindFirst(ClaimTypes.Name)?.Value;
80+
var displayName = auth.Principal.FindFirst(OAuthConstants.ClaimDisplayName)?.Value;
8181

8282
return new ValidatedFlowContext(
8383
Provider: actualScheme,
8484
Flow: flow,
8585
ExternalAccountId: externalId,
86-
ExternalAccountName: globalName ?? displayName,
87-
ExternalAccountDisplayName: displayName ?? globalName,
86+
ExternalAccountName: name ?? displayName,
87+
ExternalAccountDisplayName: displayName ?? name,
8888
Principal: auth.Principal,
8989
Properties: auth.Properties
9090
);

API/OAuth/OAuthConstants.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,5 +11,5 @@ public static class OAuthConstants
1111
public const string ItemKeyFlowType = ".FlowType";
1212

1313
public const string ClaimEmailVerified = "openshock.oauth.email_verified";
14-
public const string ClaimGlobalName = "openshock.oauth.global_name";
14+
public const string ClaimDisplayName = "openshock.oauth.display_name";
1515
}

API/Program.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,7 @@
5656
o.SaveTokens = false;
5757

5858
o.ClaimActions.MapJsonKey(OAuthConstants.ClaimEmailVerified, "verified");
59-
o.ClaimActions.MapJsonKey(OAuthConstants.ClaimGlobalName, "global_name");
59+
o.ClaimActions.MapJsonKey(OAuthConstants.ClaimDisplayName, "global_name");
6060

6161
o.Validate();
6262
});

0 commit comments

Comments
 (0)