Skip to content

Commit

Permalink
fix: authprovider error message when a user is not found in the auth.…
Browse files Browse the repository at this point in the history
…Manager
  • Loading branch information
DeepDiver1975 committed Mar 8, 2024
1 parent fc2867f commit 2764675
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions changelog/unreleased/fix-authprovider-error-message.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
Bugfix: Fix error message in authprovider if user is not found

https://github.com/cs3org/reva/pull/4567
4 changes: 2 additions & 2 deletions internal/grpc/services/authprovider/authprovider.go
Original file line number Diff line number Diff line change
Expand Up @@ -149,9 +149,9 @@ func (s *service) Authenticate(ctx context.Context, req *provider.AuthenticateRe
Status: status.NewPermissionDenied(ctx, v, "wrong password"),
}, nil
case errtypes.NotFound:
log.Debug().Str("client_id", username).Msg("unknown client id")
log.Debug().Str("username", username).Msg("unknown user")
return &provider.AuthenticateResponse{
Status: status.NewNotFound(ctx, "unknown client id"),
Status: status.NewNotFound(ctx, "unknown user"),
}, nil
default:
err = errors.Wrap(err, "authsvc: error in Authenticate")
Expand Down

0 comments on commit 2764675

Please sign in to comment.