[GitHub] storm pull request #2833: STORM-3225: Use MediaType for check

[srdo]

Github user srdo commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2833#discussion_r217882011
  
    --- Diff: 
storm-webapp/src/main/java/org/apache/storm/daemon/ui/filters/AuthorizedUserFilter.java
 ---
    @@ -79,7 +80,7 @@
         public static Response makeResponse(Exception ex, 
ContainerRequestContext request, int statusCode) {
             String callback = null;
     
    -        if 
(request.getMediaType().toString().contains("application/json")) {
    +        if (request.getMediaType() != null && 
request.getMediaType().equals(MediaType.APPLICATION_JSON_TYPE)) {
    --- End diff --
    
    Nit: I think this can be simplified to 
`MediaType.APPLICATION_JSON_TYPE.equals(request.getMediaType())`


---