diff --git a/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy b/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy index 61d5d76220a7039f60dab79b7051f9a13fefbe01..b136854f8ccd0f56916add61491d44ecb945160a 100755 --- a/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy +++ b/bigbluebutton-web/grails-app/controllers/org/bigbluebutton/web/controllers/ApiController.groovy @@ -209,7 +209,12 @@ class ApiController { // BEGIN - backward compatibility if (StringUtils.isEmpty(params.checksum)) { - invalid("checksumError", "You did not pass the checksum security check", REDIRECT_RESPONSE) + invalid("checksumError", "You did not pass the checksum security check") + return + } + + if (!paramsProcessorUtil.isChecksumSame(API_CALL, params.checksum, request.getQueryString())) { + invalid("checksumError", "You did not pass the checksum security check") return } @@ -241,11 +246,6 @@ class ApiController { return } - if (!paramsProcessorUtil.isChecksumSame(API_CALL, params.checksum, request.getQueryString())) { - invalid("checksumError", "You did not pass the checksum security check", REDIRECT_RESPONSE) - return - } - // END - backward compatibility // Do we have a checksum? If none, complain.