Load user's roles after profile-update

This commit is contained in:
Denis Arh
2021-10-11 21:03:31 +02:00
parent db7b21d855
commit 1fa84826c3
4 changed files with 12 additions and 0 deletions
+4
View File
@@ -60,6 +60,10 @@ func (h *AuthHandlers) profileProc(req *request.AuthReq) error {
// this way user service will pass us through.
user, err := h.UserService.Update(req.Context(), u)
if err == nil {
err = h.AuthService.LoadRoleMemberships(req.Context(), user)
}
if err == nil {
req.AuthUser.User = user
req.AuthUser.Save(req.Session)
+2
View File
@@ -77,6 +77,8 @@ func Test_profileFormProc(t *testing.T) {
req.PostForm.Add("handle", "handle")
req.PostForm.Add("name", "name")
authService = &authServiceMocked{}
userService = &userServiceMocked{
update: func(c context.Context, u *types.User) (*types.User, error) {
u = makeMockUser()
+1
View File
@@ -43,6 +43,7 @@ type (
ValidateTOTP(ctx context.Context, code string) (err error)
ConfigureTOTP(ctx context.Context, secret string, code string) (u *types.User, err error)
RemoveTOTP(ctx context.Context, userID uint64, code string) (u *types.User, err error)
LoadRoleMemberships(ctx context.Context, u *types.User) error
SendEmailOTP(ctx context.Context) (err error)
ConfigureEmailOTP(ctx context.Context, userID uint64, enable bool) (u *types.User, err error)
+5
View File
@@ -180,6 +180,11 @@ func (s authServiceMocked) ValidateEmailOTP(ctx context.Context, code string) (e
return s.validateEmailOTP(ctx, code)
}
func (s authServiceMocked) LoadRoleMemberships(ctx context.Context, u *types.User) error {
// no-op for now
return nil
}
//
// Mocking oauth2Service
//