root
/
spring-framework
mirror of https://github.com/spring-projects/spring-framework.git
1
0
Fork 0
Code Issues Actions 2 Packages Projects Releases Wiki Activity

Remove the catch of NPE because this issue had resolved at Jetty 12.0.2. 

Closes gh-34235
Signed-off-by: Mengqi Xu <2663479778@qq.com>
This commit is contained in:
Mengqi Xu 2025-01-10 22:08:45 +08:00 committed by Brian Clozel
parent 4d0f1729cb
commit dc98a7960a
1 changed files with 1 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
spring-websocket/src/main/java/org/springframework/web/socket/adapter/jetty/JettyWebSocketSession.java
View File

@ -183,14 +183,8 @@ public class JettyWebSocketSession extends AbstractWebSocketSession<Session> {
this.extensions = getExtensions(session); this.extensions = getExtensions(session);
if (this.user == null) { if (this.user == null) {
try {
this.user = session.getUpgradeRequest().getUserPrincipal(); this.user = session.getUpgradeRequest().getUserPrincipal();
} }
catch (NullPointerException ex) {
// Necessary until https://github.com/eclipse/jetty.project/issues/10498 is resolved
logger.error("Failure from UpgradeRequest while getting Principal", ex);
}
}
} }
private List<WebSocketExtension> getExtensions(Session session) { private List<WebSocketExtension> getExtensions(Session session) {