From d5addc98b00a8bf65b374e842bf83bf4694a4781 Mon Sep 17 00:00:00 2001
From: "Kristen.Herum" <krharum@gmail.com>
Date: Fri, 10 Jan 2025 13:07:25 +0100
Subject: [PATCH] Oppdatering for logging av identer

---
 .../logging/TestnavLogbackEncoder.java        | 19 +++++++-------
 .../logging/TestnavLogbackEncoder.java        | 25 ++++++-------------
 2 files changed, 17 insertions(+), 27 deletions(-)

diff --git a/libs/reactive-core/src/main/java/no/nav/testnav/libs/reactivecore/logging/TestnavLogbackEncoder.java b/libs/reactive-core/src/main/java/no/nav/testnav/libs/reactivecore/logging/TestnavLogbackEncoder.java
index ebaf3ca809d..dca76cb513d 100644
--- a/libs/reactive-core/src/main/java/no/nav/testnav/libs/reactivecore/logging/TestnavLogbackEncoder.java
+++ b/libs/reactive-core/src/main/java/no/nav/testnav/libs/reactivecore/logging/TestnavLogbackEncoder.java
@@ -33,8 +33,8 @@
 public class TestnavLogbackEncoder extends LogstashEncoder {
 
     // matches exactly 11 digits (\\d{11}) that are not immediately preceded ((?<!\\d)) or followed ((?!\\d)) by another digit.
-    private final Pattern identNummer = Pattern.compile("(?<!\\d)\\d{11}(?!\\d)");
-    private final Pattern bearer = Pattern.compile("Bearer [a-zA-Z0-9\\-_.]+");
+    private static final Pattern IDENT = Pattern.compile("(?<!\\d)\\d{11}(?!\\d)");
+    private static final Pattern BEARER = Pattern.compile("Bearer [a-zA-Z0-9\\-_.]+");
     @Setter
     private int maxStackTraceLength = 480;
 
@@ -113,15 +113,14 @@ private void appendStackTraceCauses(ThrowableProxy exception, StringWriter write
     }
 
     private String formatMessage(String message) {
-        message = identNummer.matcher(message).replaceAll(match -> {
-            if (match.group().charAt(2) < '4') {
-                return match.group().substring(0, 6) + "xxxxx";
-            }
-            return match.group();
-        });
 
-        message = bearer.matcher(message).replaceAll("REDACTED_BEARER");
+        message = IDENT.matcher(message).replaceAll(match ->
+
+                match.group().charAt(2) < '4' ?
+                        match.group().substring(0, 6) + "xxxxx" :
+                        match.group().substring(0, 11) + "x"
+        );
 
-        return message;
+        return BEARER.matcher(message).replaceAll("Bearer token");
     }
 }
diff --git a/libs/servlet-core/src/main/java/no/nav/testnav/libs/servletcore/logging/TestnavLogbackEncoder.java b/libs/servlet-core/src/main/java/no/nav/testnav/libs/servletcore/logging/TestnavLogbackEncoder.java
index 2158548eb68..a82cd5c197e 100644
--- a/libs/servlet-core/src/main/java/no/nav/testnav/libs/servletcore/logging/TestnavLogbackEncoder.java
+++ b/libs/servlet-core/src/main/java/no/nav/testnav/libs/servletcore/logging/TestnavLogbackEncoder.java
@@ -33,7 +33,8 @@
 public class TestnavLogbackEncoder extends LogstashEncoder {
 
     // matches exactly 11 digits (\\d{11}) that are not immediately preceded ((?<!\\d)) or followed ((?!\\d)) by another digit.
-    private final Pattern pattern = Pattern.compile("(?<!\\d)\\d{11}(?!\\d)");
+    private static final Pattern IDENT = Pattern.compile("(?<!\\d)\\d{11}(?!\\d)");
+    private static final Pattern BEARER = Pattern.compile("Bearer [a-zA-Z0-9\\-_.]+");
 
     @Setter
     private int maxStackTraceLength = 480;
@@ -113,24 +114,14 @@ private void appendStackTraceCauses(ThrowableProxy exception, StringWriter write
     }
 
     private String formatMessage(String message) {
-        var matcher = pattern.matcher(message);
 
-        if (!matcher.find()) {
-            return message;
-        }
-
-        matcher.reset();
-        var result = new StringBuilder();
+        message = IDENT.matcher(message).replaceAll(match ->
 
-        while (matcher.find()) {
-            var match = matcher.group();
-            if (match.charAt(2) == '0' || match.charAt(2) == '1') {
-                var replacement = match.substring(0, 6) + "xxxxx";
-                matcher.appendReplacement(result, replacement);
-            }
-        }
-        matcher.appendTail(result);
+                match.group().charAt(2) < '4' ?
+                        match.group().substring(0, 6) + "xxxxx" :
+                        match.group().substring(0, 11) + "x"
+        );
 
-        return result.toString();
+        return BEARER.matcher(message).replaceAll("Bearer token");
     }
 }
\ No newline at end of file