[DRILL-5745] Invalid "location" information in Drill web server

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.11.0
• Fix Version/s: 1.12.0
• Component/s: Web Server
• Labels: ready-to-commit
• Assignee: Prasad Nagaraj Subramanya
• Reporter: Paul Rogers
• Reviewer: Arina Ielchiieva
• Created: 28/Aug/17 22:08
• Updated: 25/Sep/17 17:29
• Resolved: 25/Sep/17 17:29

Description

The file ProfileResources.java has the following incorrect code line:

      this.location = "http://localhost:8047/profile/" + queryId + ".json";

This code makes three errors.

1. The "http" prefix ignores the fact that the Drillbit can have SSL enabled for the web server.
2. In a browser, "localhost" refers to the the machine running the browser. This is valid only if the browser runs on the same machine as the Drillbit, which is not, in general, true.
3. The port number is hardcoded to 8047, but it can be customized in the config file.

Therefore, most of the time, the link won't work on a production server.

Issue Links

• links to: GitHub Pull Request #948