[CAMEL-12222] ResrSwaggerServlet removes last part of context root

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Fix Version/s: 2.19.5, 2.20.3, 2.21.0
• Component/s: camel-swagger
• Estimated Complexity: Moderate
• Assignee: Claus Ibsen
• Reporter: Darius Cooper
• Created: 02/Feb/18 00:53
• Updated: 13/Feb/18 14:32
• Resolved: 13/Feb/18 14:32

Description

The method within RestSwaggerServlet, looks for the last "/" in the context root, and drops anything after that slash.

private String translateContextPath(HttpServletRequest request) {
  String path = request.getContextPath();
  if (path.isEmpty() || path.equals("/")) {
      return "";
  } else {
     int idx = path.lastIndexOf("/");
     if (idx > 0) {
         return path.substring(0, idx);
     }
  }
  return path;
}

Even though this has been deprecated, it would be good to fix it. 

An approach could be to add a servlet init parameter named "translateContextPath" which would default to true, and would work exactly as it does today by default. if set to false, it would return whatever value is in request.getContextPath() , with no translation.

Also, to avoid the NPE thrown, pass in an empty RestConfiguration to RestSwaggerSupport.renderResourceListing() 

(See: https://issues.apache.org/jira/browse/CAMEL-12038) 

I plan on working on this. Creating a Jira to get feedback on the approach

Issue Links

• links to: GitHub Pull Request #2220
• links to: GitHub Pull Request #2220