Revert to combining the handleClearListValCache and handleClearListSt…

…atCache under a single try block (the original fix to vulnerability).
dtcenter · Aug 19, 2024 · b968e96 · b968e96
1 parent 06f5693
commit b968e96
Showing 1 changed file with 1 addition and 9 deletions.
diff --git a/java/edu/ucar/metviewer/MVServlet.java b/java/edu/ucar/metviewer/MVServlet.java
@@ -1294,20 +1294,12 @@ public void doPost(HttpServletRequest request, HttpServletResponse response) {
             strResp.append(MVUtil.domSourceToString(docResp));
             try {
               handleClearListValCache();
+              handleClearListStatCache();
             } catch (ParserConfigurationException e) {
               logger.error(e.getMessage());
             } catch (Exception e){
               logger.error(e.getMessage());
-            } finally{
-              return;
             }
-
-            try{
-              handleClearListStatCache();
-            } catch (ParserConfigurationException e) {
-              logger.error(e.getMessage());
-            }
-
           }
 
           //  <date> tag, which is used to prevent caching