[STORM-2797] LogViewer worker logs broken on Windows

• Type: Bug
• Status: Closed
• Priority: Minor
• Resolution: Abandoned
• Affects Version/s: 1.x
• Component/s: storm-webapp
• Labels: pull-request-available
• Environment: Windows
• Assignee: Unassigned
• Reporter: Lawrence Craft
• Created: 03/Nov/17 23:58
• Updated: 02/Jul/24 09:54
• Resolved: 02/Jul/24 09:54

Description

LogViewer worker logs are broken on Windows. Attempting to access the log (e.g. http://localhost:8000/log?file=word-topo-5-1509750559%5C6701%5Cworker.log) leads to a 500 Server Error.

I've attached the LogViewer logs which show the stack trace. The issue is pretty clear from the log: on line 123 of logviewer.clj, the path is split using the path separator as a regex. This is fine on Posix systems as / is a normal character in regex; however, on Windows, backslash is the path separator. As this is also the regex escape character, it is not a valid regular expression.

Issue Links

• links to: GitHub Pull Request #2414