From ca312900966184bf09b9adb4274dc35d08660b33 Mon Sep 17 00:00:00 2001
From: Philip Durbin <philip_durbin@harvard.edu>
Date: Wed, 30 Aug 2023 17:02:10 -0400
Subject: [PATCH] reduce logging for download file guestbook/terms bug #9806

---
 doc/release-notes/9340-payara5to6.md             | 16 ++++++++++++++++
 .../container-base/src/main/docker/Dockerfile    |  2 ++
 scripts/installer/as-setup.sh                    |  3 +++
 3 files changed, 21 insertions(+)

diff --git a/doc/release-notes/9340-payara5to6.md b/doc/release-notes/9340-payara5to6.md
index 68162ef7598..339086b9ab2 100644
--- a/doc/release-notes/9340-payara5to6.md
+++ b/doc/release-notes/9340-payara5to6.md
@@ -130,3 +130,19 @@
    `sudo -u dataverse /usr/local/payara6/bin/asadmin stop-domain`
 
    `sudo -u dataverse /usr/local/payara6/bin/asadmin start-domain`
+
+1. Reducing logging related to third party code used with Terms of Use/Guestbook
+
+   Some third party library code is used in the UI of the application and unfortunately, in this release, it seems to be leading to excessive logging, despite everything appearing to work properly. For example, downloading a file with Terms of Use/Guestbook enabled works, but a very long exception stack trace ends up in server.log (look for `ViewHandlingStrategyNotFoundException`).
+
+   To supress these messages, the following line can be added to `/usr/local/payara6/glassfish/domains/domain1/config/logging.properties`
+
+   `jakarta.enterprise.resource.webcontainer.faces.level=OFF`
+
+   Alternatively, you can run this (it does the same thing):
+
+   `asadmin set-log-levels jakarta.enterprise.resource.webcontainer.faces=OFF`
+
+   A Payara restart should not be required.
+
+   If you'd like additional details on this bug, please see issue #9806.
diff --git a/modules/container-base/src/main/docker/Dockerfile b/modules/container-base/src/main/docker/Dockerfile
index 97aa4cd2792..09884c0434d 100644
--- a/modules/container-base/src/main/docker/Dockerfile
+++ b/modules/container-base/src/main/docker/Dockerfile
@@ -155,6 +155,8 @@ RUN <<EOF
     ${ASADMIN} create-jvm-options "-XX\:HeapDumpPath=\${ENV=DUMPS_DIR}"
     # Set logging to console only for containers
     ${ASADMIN} set-log-attributes com.sun.enterprise.server.logging.GFFileHandler.logtoFile=false \
+    # Workaround for excessive logging when downloading file if guestbook: https://github.com/IQSS/dataverse/issues/9806
+    ${ASADMIN} set-log-levels jakarta.enterprise.resource.webcontainer.faces=OFF
 
     ### PRODUCTION READINESS
     ${ASADMIN} create-jvm-options '-XX\:+UseG1GC'
diff --git a/scripts/installer/as-setup.sh b/scripts/installer/as-setup.sh
index fc5b378cff5..5268d4b38e5 100755
--- a/scripts/installer/as-setup.sh
+++ b/scripts/installer/as-setup.sh
@@ -119,6 +119,9 @@ function preliminary_setup()
   # Workaround for FISH-7722: Failed to deploy war with @Stateless https://github.com/payara/Payara/issues/6337
   ./asadmin $ASADMIN_OPTS create-jvm-options --add-opens=java.base/java.io=ALL-UNNAMED
 
+  # Workaround for excessive logging when downloading file if guestbook: https://github.com/IQSS/dataverse/issues/9806
+  ./asadmin $ASADMIN_OPTS set-log-levels jakarta.enterprise.resource.webcontainer.faces=OFF
+
   # enable comet support
   ./asadmin $ASADMIN_OPTS set server-config.network-config.protocols.protocol.http-listener-1.http.comet-support-enabled="true"