From d30fe65594672bd895d68c823a3a4294f95de9f3 Mon Sep 17 00:00:00 2001
From: Jens Hardings <jhp@moit.cl>
Date: Mon, 8 Jul 2024 20:44:05 -0400
Subject: [PATCH] allow tracking of artifact hits for rest paths (defaults to
 false)

---
 conf/MoquiDevConf.xml        | 3 ++-
 conf/MoquiLoadTestConf.xml   | 2 +-
 conf/MoquiProductionConf.xml | 2 +-
 3 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/conf/MoquiDevConf.xml b/conf/MoquiDevConf.xml
index c74876868..2fb798e14 100644
--- a/conf/MoquiDevConf.xml
+++ b/conf/MoquiDevConf.xml
@@ -51,7 +51,7 @@
         <cache name="l10n.message" expire-time-idle="600"/>
     </cache-list>
     
-    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/rest') || pathInfo?.startsWith('/status')">
+    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/status')">
         <!-- For development, track everything! It'll run slow through... -->
         <artifact-stats type="AT_XML_SCREEN" persist-bin="true" persist-hit="true"/>
         <artifact-stats type="AT_XML_SCREEN_CONTENT" persist-bin="true" persist-hit="true"/>
@@ -59,6 +59,7 @@
         <!-- NOTE: entity-implicit and entity-auto services never persist hits -->
         <artifact-stats type="AT_SERVICE" persist-bin="true" persist-hit="true"/>
         <artifact-stats type="AT_ENTITY" persist-bin="true"/>
+        <artifact-stats type="AT_REST_PATH" persist-bin="true" persist-hit="true"/>
     </server-stats>
 
     <webapp-list>
diff --git a/conf/MoquiLoadTestConf.xml b/conf/MoquiLoadTestConf.xml
index 34c4504e4..04db9964c 100644
--- a/conf/MoquiLoadTestConf.xml
+++ b/conf/MoquiLoadTestConf.xml
@@ -5,7 +5,7 @@
     https://github.com/moqui/moqui-framework/blob/master/framework/src/main/resources/MoquiDefaultConf.xml -->
 <moqui-conf xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://moqui.org/xsd/moqui-conf-3.xsd">
 
-    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/rest') || pathInfo?.startsWith('/status')"/>
+    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/status')"/>
 
     <!-- for load test turn off tarpit -->
     <artifact-execution-facade>
diff --git a/conf/MoquiProductionConf.xml b/conf/MoquiProductionConf.xml
index 2177fbb98..cc2ed4794 100644
--- a/conf/MoquiProductionConf.xml
+++ b/conf/MoquiProductionConf.xml
@@ -5,7 +5,7 @@
     https://github.com/moqui/moqui-framework/blob/master/framework/src/main/resources/MoquiDefaultConf.xml -->
 <moqui-conf xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="http://moqui.org/xsd/moqui-conf-3.xsd">
 
-    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/rest') || pathInfo?.startsWith('/status')"/>
+    <server-stats stats-skip-condition="pathInfo?.startsWith('/rpc') || pathInfo?.startsWith('/status')"/>
 
     <!-- NOTE: using the environment variable is relatively secure in a container environment, but for more security set it here instead -->
     <entity-facade crypt-pass="${entity_ds_crypt_pass}" query-stats="true">