Improve logging (Unidata#472)

* Make sure edal-java style loading code logs errors * Add extra info to log
WeatherGod · Apr 18, 2024 · 7b57857 · 7b57857
1 parent 4fc5ea9
commit 7b57857
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/tds/src/main/java/thredds/server/config/TdsConfigMapper.java b/tds/src/main/java/thredds/server/config/TdsConfigMapper.java
@@ -212,6 +212,7 @@ static void load(WmsConfigBean wmsConfig, TdsContext tdsContext) {
           getValueFromThreddsConfigOrDefault(WMS_PALETTE_LOCATION_DIR, defaultPaletteLocation);
       wmsConfig.setPaletteLocationDir(paletteLocation);
       try {
+        startupLog.info("Loading custom WMS palette files from " + paletteLocation);
         ColourPalette.addPaletteDirectory(new File(paletteLocation));
       } catch (FileNotFoundException e) {
         // If there is an error adding a custom palette, it is logged in the server startup log by edal-java.
@@ -226,6 +227,7 @@ static void load(WmsConfigBean wmsConfig, TdsContext tdsContext) {
       final String stylesLocation = getValueFromThreddsConfigOrDefault(WMS_STYLES_LOCATION_DIR, defaultStylesLocation);
       wmsConfig.setStylesLocationDir(stylesLocation);
       try {
+        startupLog.info("Loading custom WMS style files from " + stylesLocation);
         SldTemplateStyleCatalogue.getStyleCatalogue().addStylesInDirectory(new File(stylesLocation));
       } catch (FileNotFoundException e) {
         if (!stylesLocation.equals(defaultStylesLocation)) {

diff --git a/tds/src/main/webapp/WEB-INF/classes/log4j2.xml b/tds/src/main/webapp/WEB-INF/classes/log4j2.xml
@@ -176,8 +176,8 @@
       <appender-ref ref="threddsServlet"/>
     </logger>
 
-    <!-- log issues related to loading custom color palettes -->
-    <logger name="uk.ac.rdg.resc.edal.graphics.utils.ColourPalette" level="warn" additivity="false">
+    <!-- log issues related to loading custom color palettes and styles -->
+    <logger name="uk.ac.rdg.resc.edal.graphics.utils" level="warn" additivity="false">
       <appender-ref ref="serverStartup"/>
     </logger>