openvidu-server: avoid NullPointerException on empty getStringParam()

pull/375/head
pabloFuente 2019-09-19 09:44:27 +02:00
parent d393b1b0e8
commit a027cb46d1
1 changed files with 5 additions and 3 deletions

View File

@ -167,7 +167,7 @@ public class RpcHandler extends DefaultJsonRpcHandler<JsonObject> {
} }
} }
public void joinRoom(RpcConnection rpcConnection, Request<JsonObject> request) { private void joinRoom(RpcConnection rpcConnection, Request<JsonObject> request) {
String sessionId = getStringParam(request, ProtocolElements.JOINROOM_ROOM_PARAM); String sessionId = getStringParam(request, ProtocolElements.JOINROOM_ROOM_PARAM);
String token = getStringParam(request, ProtocolElements.JOINROOM_TOKEN_PARAM); String token = getStringParam(request, ProtocolElements.JOINROOM_TOKEN_PARAM);
@ -661,7 +661,9 @@ public class RpcHandler extends DefaultJsonRpcHandler<JsonObject> {
public static String getStringParam(Request<JsonObject> request, String key) { public static String getStringParam(Request<JsonObject> request, String key) {
if (request.getParams() == null || request.getParams().get(key) == null) { if (request.getParams() == null || request.getParams().get(key) == null) {
throw new RuntimeException("Request element '" + key + "' is missing in method '" + request.getMethod() throw new RuntimeException("Request element '" + key + "' is missing in method '" + request != null
? request.getMethod()
: "[NO REQUEST OBJECT]"
+ "'. CHECK THAT 'openvidu-server' AND 'openvidu-browser' SHARE THE SAME VERSION NUMBER"); + "'. CHECK THAT 'openvidu-server' AND 'openvidu-browser' SHARE THE SAME VERSION NUMBER");
} }
return request.getParams().get(key).getAsString(); return request.getParams().get(key).getAsString();