Fixed log injection into method

pagopa · Oct 14, 2024 · 7486ed1 · 7486ed1
1 parent 7d302cb
commit 7486ed1
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/...rding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java b/...rding-ms/src/main/java/it/pagopa/selfcare/onboarding/controller/OnboardingController.java
@@ -433,13 +433,12 @@ public Uni<Response> update(@PathParam(value = "onboardingId") String onboarding
   public Uni<Response> updateRecipientCodeByOnboardingId(
       @PathParam(value = "onboardingId") String onboardingId,
       @QueryParam(value = "recipientCode") String recipientCode) {
-    String fixedRecipientCode = recipientCode.trim().replace("\n", "").replace("\r", "");
     Onboarding onboarding = new Onboarding();
     Billing billing = new Billing();
-    billing.setRecipientCode(fixedRecipientCode);
+    billing.setRecipientCode(recipientCode.trim());
     onboarding.setBilling(billing);
     log.trace("update RecipientCode start");
-    log.debug("Onboarding id {} and recipientCode {}", onboardingId, fixedRecipientCode);
+    log.debug("Onboarding id {} and recipientCode {}", onboardingId, recipientCode.replace("\n", "").replace("\r", ""));
     return onboardingService
         .updateOnboarding(onboardingId, onboarding)
         .map(ignore -> Response.status(HttpStatus.SC_NO_CONTENT).build())