reta commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572368151
##########
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##########
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchange
ex) {
public Principal getUserPrincipal() {
try {
return req.getUserPrincipal();
- } catch (Exception ex) {
- return null;
+ } catch (Exception e) {
Review Comment:
@rzo1 thanks for catching it, @ffang do you have an idea why `request` could
be `null` in `ServletApiRequest`? (may be we have problem somewhere with Jetty
12 integration)
```
at
org.eclipse.jetty.server.Request.getAuthenticationState(Request.java:949)
at
org.eclipse.jetty.security.AuthenticationState.getAuthenticationState(AuthenticationState.java:45)
at
org.eclipse.jetty.ee10.servlet.ServletApiRequest.getAuthentication(ServletApiRequest.java:254)
at
org.eclipse.jetty.ee10.servlet.ServletApiRequest.getUndeferredAuthentication(ServletApiRequest.java:259)
at
org.eclipse.jetty.ee10.servlet.ServletApiRequest.getUserPrincipal(ServletApiRequest.java:478)
at
org.apache.cxf.transport.http.AbstractHTTPDestination$2.getUserPrincipal(AbstractHTTPDestination.java:412)
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
