Update feishu provider claims.

aspnet-contrib / AspNet.Security.OAuth.Providers

OAuth 2.0 social authentication providers for ASP.NET Core

Apache License 2.0

2.34k stars 533 forks source link

Update feishu provider claims. #821

Closed vicenteyu closed 6 months ago

vicenteyu commented 6 months ago

Add new claims:

Sub;
Name;
Picture;
OpenId;
EnName;
TenantKey;
AvatarUrl;
AvatarThumb;
AvatarMiddle;
AvatarBig;
Email;
UserId;
EmployeeNo;
Mobile;

vicenteyu commented 6 months ago

Adding more claims will make it easier to use.

martincostello commented 6 months ago

I get that it makes it "easier", but it also means mapping more things that an integration might actually use.

I'm not convinced there's a need to add all these additional mappings in the common/default case of just logging in a user and knowing their identity.

vicenteyu commented 6 months ago

I get that it makes it "easier", but it also means mapping more things that an integration might actually use.

I'm not convinced there's a need to add all these additional mappings in the common/default case of just logging in a user and knowing their identity.

Maybe making it an expansion pack is a more appropriate way, thank you.