[STORM-2814] Logviewer HTTP server should return 403 instead of 200 if the user is unauthorized

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.0.0, 1.2.0, 1.1.2
• Labels: pull-request-available
• Assignee: Ethan Li
• Reporter: Ethan Li
• Created: 14/Nov/17 18:28
• Updated: 16/Nov/17 04:44
• Resolved: 15/Nov/17 23:30

Description

    public static Response buildResponseUnautohrizedUser(String user) {
        String entity = buildUnauthorizedUserHtml(user);
        return Response.status(OK)
                .entity(entity)
                .type(MediaType.TEXT_HTML_TYPE)
                .build();
    }

It returns OK which is confusing.

Issue Links

• links to: GitHub Pull Request #2420
• links to: GitHub Pull Request #2422