Handle log injection code check

eclipse-tractusx · Jul 29, 2024 · a4c9d09 · a4c9d09
1 parent 7639c79
commit a4c9d09
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/...rc/main/java/org/eclipse/tractusx/sde/pcfexchange/service/impl/ProxyRequestInterface.java b/...rc/main/java/org/eclipse/tractusx/sde/pcfexchange/service/impl/ProxyRequestInterface.java
@@ -103,7 +103,7 @@ public void requestToProviderForPCFValue(String productId, StringBuilder reponse
 			log.error("FeignRequest requestToProviderForPCFValue:" + e.request());
 			String error= StringUtils.isBlank(e.contentUTF8()) ? e.getMessage() : e.contentUTF8();
 			String errorMsg= "Unable to request to provider '"+ dataset.getConnectorOfferUrl()+"' for '"+productId+"' product PCF value beacuse error in remote service execution";
-			log.error("FeignException requestToProviderForPCFValue: " + errorMsg + ", because: " +error);
+			log.error(LogUtil.encode("FeignException requestToProviderForPCFValue: " + errorMsg + ", because: " +error));
 			reponseMap.append(errorMsg);
 		}
 	}

diff --git a/...odules/pcf/src/main/java/org/eclipse/tractusx/sde/submodels/pcf/PCFEDCUsecaseHandler.java b/...odules/pcf/src/main/java/org/eclipse/tractusx/sde/submodels/pcf/PCFEDCUsecaseHandler.java
@@ -59,6 +59,7 @@ public class PCFEDCUsecaseHandler extends Step implements EDCUsecaseStep {
     private final EDCGateway edcGateway;
 
     @SneakyThrows
+    @Override
     public ObjectNode run(Integer rowNumber, ObjectNode objectNode, String processId, PolicyModel policy) {
 
         Map<String, String> output = new HashMap<>();
@@ -127,7 +128,7 @@ public ObjectNode run(Integer rowNumber, ObjectNode objectNode, String processId
 
     }
 
-
+    @Override
     public void delete(Integer rowIndex, JsonObject jsonObject, String delProcessId, String refProcessId) {
         log.warn("No need to delete EDC asset for PCF exchange");