Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

UserContext中Roles使用数组或字典，不使用IdentityRole #178

Closed
qinxh opened this issue Aug 6, 2022 · 1 comment · Fixed by #180 or #192
Closed

UserContext中Roles使用数组或字典，不使用IdentityRole #178

qinxh opened this issue Aug 6, 2022 · 1 comment · Fixed by #180 or #192
Assignees
@MayueCif
Milestone
0.6.0

Comments

@qinxh
Copy link
Contributor

qinxh commented Aug 6, 2022

UserContext中Roles使用数组或字典，这样对于使用者集成Masa会更友好。
如果我们直接早已封装好ICurrentUser，不过我们的Roles使用的是数组 string[] Roles { get; }
目前升级0.5，为了兼容MASA，底层封装也得同步调整，必须引入Masa.BuildingBlocks.Identity.IdentityModel。
如果UserContext中Roles使用数组或字典，像这样情况改动就很小，只需在应用中设置IdentityClaimOptions即可。
@zhenlei520
Copy link
Contributor

zhenlei520 commented Aug 6, 2022 •
edited
Loading

我们计划在0.6.0安排时间重构IdentityModel，使得ICurrentUser可以支持自定义UserModel，目前实现上不支持自定义UserModel，Roles类型的问题后面会与对应的开发沟通一下，这里应该是不需要名称

@zhenlei520 zhenlei520 added this to the 0.6.0 milestone Aug 6, 2022
@zhenlei520 zhenlei520 assigned zhenlei520 and MayueCif and unassigned zhenlei520 Aug 6, 2022
This was linked to pull requests Aug 9, 2022
feat:user visit fix #180
Merged
refactor(Identity): Support custom User Model #192
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@MayueCif MayueCif

Labels
None yet
Projects
None yet
Milestone
0.6.0
3 participants
@MayueCif @zhenlei520 @qinxh