ddanielr commented on code in PR #5729:
URL: https://github.com/apache/accumulo/pull/5729#discussion_r2195754255
##########
server/monitor/src/main/java/org/apache/accumulo/monitor/view/WebViews.java:
##########
@@ -93,12 +93,21 @@ private void addExternalResources(Map<String,Object> model)
{
}
private Map<String,Object> getModel() {
+ AccumuloConfiguration conf = monitor.getContext().getConfiguration();
+ String rootContext = conf.get(Property.MONITOR_ROOT_CONTEXT);
+ if (!rootContext.startsWith("/")) {
Review Comment:
The slashes are being added because the variable is being concatenated to
string values in the FreeMarker templates. If the slashes are not set
correctly, then it can cause the paths to be incorrectly generated.
Alternatively these checks could be preconditions and force the properties
to be set correctly on monitor startup (kill the monitor otherwise) vs
modifying the values at runtime.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
