Skip to content

Commit

Permalink
add a TODO about better errors for the client IQSS#9229
Browse files Browse the repository at this point in the history
  • Loading branch information
pdurbin authored May 17, 2023
1 parent 2885f68 commit 9c7fe07
Showing 1 changed file with 3 additions and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,9 @@ public User findUserFromRequest(ContainerRequestContext containerRequestContext)
return authUser;
} else {
// a valid Token was presented, but we have no associated user account.
logger.log(Level.WARNING, "Bearer token detected, OIDC provider {0} validated Token but no linked UserAccount", userInfo.getUserRepoId());
logger.log(Level.WARNING, "Bearer token detected, OIDC provider {0} validated Token but no linked UserAccount", userInfo.getUserRepoId());
// TODO: Instead of returning null, we should throw a meaningful error to the client.
// Probably this will be a wrapped auth error response with an error code and a string describing the problem.
return null;
}
}
Expand Down

0 comments on commit 9c7fe07

Please sign in to comment.