Skip to content

Commit 9326690

Browse files
authored
fix: handle null close code and reason in WebSocketRoute (#1886)
1 parent 4804007 commit 9326690

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

playwright/src/main/java/com/microsoft/playwright/impl/WebSocketRouteImpl.java

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -160,9 +160,9 @@ protected void handleEvent(String event, JsonObject params) {
160160
sendMessageAsync("sendToPage", messageParams);
161161
}
162162
} else if ("closePage".equals(event)) {
163-
int code = params.get("code").getAsInt();
164-
String reason = params.get("reason").getAsString();
165-
boolean wasClean = params.get("wasClean").getAsBoolean();
163+
Integer code = params.has("code") ? params.get("code").getAsInt() : null;
164+
String reason = params.has("reason") ? params.get("reason").getAsString() : null;
165+
boolean wasClean = params.has("wasClean") && params.get("wasClean").getAsBoolean();
166166
if (onPageClose != null) {
167167
onPageClose.accept(code, reason);
168168
} else {
@@ -173,9 +173,9 @@ protected void handleEvent(String event, JsonObject params) {
173173
sendMessageAsync("closeServer", closeParams);
174174
}
175175
} else if ("closeServer".equals(event)) {
176-
int code = params.get("code").getAsInt();
177-
String reason = params.get("reason").getAsString();
178-
boolean wasClean = params.get("wasClean").getAsBoolean();
176+
Integer code = params.has("code") ? params.get("code").getAsInt() : null;
177+
String reason = params.has("reason") ? params.get("reason").getAsString() : null;
178+
boolean wasClean = params.has("wasClean") && params.get("wasClean").getAsBoolean();
179179
if (onServerClose != null) {
180180
onServerClose.accept(code, reason);
181181
} else {

0 commit comments

Comments
 (0)