From 4ff246ce80b20e49a12bd08111912ecd506024f5 Mon Sep 17 00:00:00 2001
From: manuraf <manuel.rafeli@pagopa.it>
Date: Tue, 8 Oct 2024 17:02:16 +0200
Subject: [PATCH] add Encode.forJava

---
 .../dashboard/connector/rest/ProductConnectorImpl.java         | 3 ++-
 .../it/pagopa/selfcare/dashboard/core/ProductServiceImpl.java  | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/connector/rest/src/main/java/it/pagopa/selfcare/dashboard/connector/rest/ProductConnectorImpl.java b/connector/rest/src/main/java/it/pagopa/selfcare/dashboard/connector/rest/ProductConnectorImpl.java
index 8dbe21b1..381cf2f4 100644
--- a/connector/rest/src/main/java/it/pagopa/selfcare/dashboard/connector/rest/ProductConnectorImpl.java
+++ b/connector/rest/src/main/java/it/pagopa/selfcare/dashboard/connector/rest/ProductConnectorImpl.java
@@ -9,6 +9,7 @@
 import it.pagopa.selfcare.product.entity.ProductRoleInfo;
 import it.pagopa.selfcare.product.service.ProductService;
 import lombok.extern.slf4j.Slf4j;
+import org.owasp.encoder.Encode;
 import org.springframework.stereotype.Service;
 import org.springframework.util.Assert;
 
@@ -36,7 +37,7 @@ public List<Product> getProducts() {
 
     @Override
     public Map<PartyRole, ProductRoleInfo> getProductRoleMappings(String productId, String institutionType) {
-        log.debug(LogUtils.CONFIDENTIAL_MARKER, "getProduct productId = {}, institutionType = {}", productId, institutionType);
+        log.debug(LogUtils.CONFIDENTIAL_MARKER, "getProduct productId = {}, institutionType = {}", Encode.forJava(productId), Encode.forJava(institutionType));
         Assert.hasText(productId, "A productId is required");
         Product product = productService.getProduct(productId);
         Map<PartyRole, ProductRoleInfo> result = product != null ? product.getRoleMappings(institutionType) : null;
diff --git a/core/src/main/java/it/pagopa/selfcare/dashboard/core/ProductServiceImpl.java b/core/src/main/java/it/pagopa/selfcare/dashboard/core/ProductServiceImpl.java
index 8f5e2cd4..7a774316 100644
--- a/core/src/main/java/it/pagopa/selfcare/dashboard/core/ProductServiceImpl.java
+++ b/core/src/main/java/it/pagopa/selfcare/dashboard/core/ProductServiceImpl.java
@@ -4,6 +4,7 @@
 import it.pagopa.selfcare.onboarding.common.PartyRole;
 import it.pagopa.selfcare.product.entity.ProductRoleInfo;
 import lombok.extern.slf4j.Slf4j;
+import org.owasp.encoder.Encode;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.stereotype.Service;
 import org.springframework.util.Assert;
@@ -26,7 +27,7 @@ class ProductServiceImpl implements ProductService {
     @Override
     public Map<PartyRole, ProductRoleInfo> getProductRoles(String productId, String institutionType) {
         log.trace("getProductRoles start");
-        log.debug("getProductRoles productId = {}, institutionType = {}", productId, institutionType);
+        log.debug("getProductRoles productId = {}, institutionType = {}", Encode.forJava(productId), Encode.forJava(institutionType));
         Assert.hasText(productId, "A Product id is required");
 
         Map<PartyRole, ProductRoleInfo> productRoleMappings = productsConnector.getProductRoleMappings(productId, institutionType);