[ https://issues.apache.org/jira/browse/CAMEL-12222?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Claus Ibsen resolved CAMEL-12222. --------------------------------- Resolution: Fixed Assignee: Claus Ibsen Fix Version/s: 2.21.0 2.20.3 2.19.5 Thanks for the PR. I fixed the checkstyle issue http://camel.apache.org/building.html > ResrSwaggerServlet removes last part of context root > ---------------------------------------------------- > > Key: CAMEL-12222 > URL: https://issues.apache.org/jira/browse/CAMEL-12222 > Project: Camel > Issue Type: Bug > Components: camel-swagger > Reporter: Darius Cooper > Assignee: Claus Ibsen > Priority: Minor > Fix For: 2.19.5, 2.20.3, 2.21.0 > > > The method within RestSwaggerServlet, looks for the last "/" in the context > root, and drops anything after that slash. > {code:java} > 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; > }{code} > 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 -- This message was sent by Atlassian JIRA (v7.6.3#76005)