CHANGELOG.md

+## [17.3.1](https://gitlab.ics.muni.cz/perun/perun-proxyidp/v1/OpenID-Connect-Java-Spring-Server/compare/v17.3.0...v17.3.1) (2024-03-13)
+
+
+### Bug Fixes
+
+* 🐛 Add switch for LogRequests filter ([a2d190a](https://gitlab.ics.muni.cz/perun/perun-proxyidp/v1/OpenID-Connect-Java-Spring-Server/commit/a2d190ae19789b554a55f2f770e790dcfe658ff1))
+
 # [17.3.0](https://gitlab.ics.muni.cz/perun/perun-proxyidp/v1/OpenID-Connect-Java-Spring-Server/compare/v17.2.2...v17.3.0) (2024-03-13)
 
 

perun-oidc-server-webapp/pom.xml

@@ -21,7 +21,7 @@
 	<parent>
 		<groupId>cz.muni.ics</groupId>
 		<artifactId>perun-oidc-parent</artifactId>
-		<version>17.3.0</version>
+		<version>17.3.1</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 

perun-oidc-server-webapp/src/main/webapp/WEB-INF/user-context.xml

@@ -37,6 +37,7 @@
 				<prop key="perun.adapter.callFallback">true</prop>
 				<prop key="fill.missing.user.attrs">true</prop>
 				<prop key="perun.adapter.primary">RPC</prop>
+				<prop key="logRequestsEnabled">false</prop>
 				<!-- RPC -->
 				<prop key="perun.rpc.enabled">true</prop>
 				<prop key="perun.rpc.url">https://perun.elixir-czech.cz/krb/rpc</prop>
@@ -474,6 +475,7 @@
 		<property name="krbTokenExchangeApiPassword" value="${token-exchange.kerberos.translator.password}"/>
 		<property name="krbTokenExchangeRequiredScopes" value="#{'${token-exchange.kerberos.requiredScopes}'.split('\s*,\s*')}"/>
 		<property name="requesterIdPrefix" value="${saml.requester-id.prefix}"/>
+		<property name="logRequestsEnabled" value="${logRequestsEnabled}"/>
 	</bean>
 
 	<bean id="facilityAttrsConfig" class="cz.muni.ics.oidc.server.configurations.FacilityAttrsConfig">

perun-oidc-server-webapp/src/main/webapp/WEB-INF/web-context.xml

@@ -116,7 +116,9 @@
         <security:expression-handler ref="oauthExpressionHandler" />
     </security:global-method-security>
 
-    <bean id="logRequestFilter" class="cz.muni.ics.oidc.web.LogRequestFilter"/>
+    <bean id="logRequestFilter" class="cz.muni.ics.oidc.web.LogRequestFilter">
+        <constructor-arg name="config" ref="perunOidcConfig"/>
+    </bean>
 
     <!-- Token endpoint -->
     <security:http pattern="/token"

perun-oidc-server/pom.xml

@@ -22,7 +22,7 @@
 	<parent>
 		<groupId>cz.muni.ics</groupId>
 		<artifactId>perun-oidc-parent</artifactId>
-		<version>17.3.0</version>
+		<version>17.3.1</version>
 		<relativePath>../pom.xml</relativePath>
 	</parent>
 

perun-oidc-server/src/main/java/cz/muni/ics/oidc/server/configurations/PerunOidcConfig.java

@@ -92,6 +92,8 @@ public class PerunOidcConfig implements InitializingBean {
 	@Autowired
 	private Properties coreProperties;
 
+	private boolean logRequestsEnabled;
+
 	public String getPerunOIDCVersion() {
 		if (perunOIDCVersion == null) {
 			perunOIDCVersion = readPomVersion(OIDC_POM_FILE);

perun-oidc-server/src/main/java/cz/muni/ics/oidc/web/LogRequestFilter.java

@@ -27,30 +27,38 @@ public class LogRequestFilter extends OncePerRequestFilter {
             "HEADERS: {}" +
             WRAPPER;
 
+    private final boolean enabled;
+
+    public LogRequestFilter(PerunOidcConfig config) {
+        enabled = config.isLogRequestsEnabled();
+    }
+
     @Override
     protected void doFilterInternal(final HttpServletRequest req,
                                     HttpServletResponse response,
                                     FilterChain filterChain)
             throws ServletException, IOException
     {
-        log.trace(MSG, req.getMethod(), req.getRequestURL(),
-                Collections.list(req.getParameterNames())
-                        .stream()
-                        .collect(
-                                Collectors.toMap(
-                                        parameter -> parameter,
-                                        parameter -> List.of(req.getParameterValues((String) parameter))
-                                )
-                        ),
-                Collections.list(req.getHeaderNames())
-                        .stream()
-                        .collect(
-                                Collectors.toMap(
-                                        header -> header,
-                                        header -> List.of(req.getHeaders((String)header))
-                                )
-                        )
-        );
+        if (enabled) {
+            log.trace(MSG, req.getMethod(), req.getRequestURL(),
+                    Collections.list(req.getParameterNames())
+                            .stream()
+                            .collect(
+                                    Collectors.toMap(
+                                            parameter -> parameter,
+                                            parameter -> List.of(req.getParameterValues((String) parameter))
+                                    )
+                            ),
+                    Collections.list(req.getHeaderNames())
+                            .stream()
+                            .collect(
+                                    Collectors.toMap(
+                                            header -> header,
+                                            header -> List.of(req.getHeaders((String) header))
+                                    )
+                            )
+            );
+        }
 
         filterChain.doFilter(req, response);
     }

pom.xml

@@ -3,7 +3,7 @@
 	<modelVersion>4.0.0</modelVersion>
 	<groupId>cz.muni.ics</groupId>
 	<artifactId>perun-oidc-parent</artifactId>
-	<version>17.3.0</version>
+	<version>17.3.1</version>
 	<packaging>pom</packaging>
 
 	<modules>