Improve the revisions with more data.

Author: flexable777

src/main/kotlin/no/nav/klage/oppgave/eventlisteners/OurRevisionListener.kt

@@ -26,19 +26,51 @@ class OurRevisionListener(
     override fun newRevision(revisionEntity: Any?) {
         revisionEntity as OurRevision
 
-        try {
+        val request = try {
             val requestAttributes = RequestContextHolder.getRequestAttributes()
             if (requestAttributes != null) {
                 val request = (requestAttributes as ServletRequestAttributes).request
-                revisionEntity.actor = tokenUtil.getIdent()
-                revisionEntity.request = request.method + " " + request.requestURI
+                request.method + " " + request.requestURI
             } else {
-                revisionEntity.actor = systembrukerIdent
+                null
             }
-            revisionEntity.traceId = Span.current().spanContext.traceId
         } catch (e: Exception) {
-            logger.warn("Failed to set correct actor and/or request on revision entity. Setting 'unknown'.", e)
-            revisionEntity.actor = "unknown"
+            logger.debug("No request found to set on revision entity. Setting to null.", e)
+            null
         }
+
+        val actor = if (request == null) {
+            systembrukerIdent
+        } else {
+            val navIdentFromToken = try {
+                tokenUtil.getIdent()
+            } catch (e: Exception) {
+                logger.debug("No NAVIdent found in token.", e)
+                null
+            }
+            val callingApplication = try {
+                tokenUtil.getCallingApplication()
+            } catch (e: Exception) {
+                logger.warn("Failed to get calling application from token.", e)
+                null
+            }
+
+            if (navIdentFromToken == null && callingApplication == null) {
+                logger.warn("Neither NAVIdent nor calling application could be found from token. Setting 'unknown'.")
+            }
+
+            navIdentFromToken ?: callingApplication ?: "unknown"
+        }
+
+        val traceId = try {
+            Span.current().spanContext.traceId
+        } catch (e: Exception) {
+            logger.warn("Failed to set traceId on revision entity. Setting 'unknown'.", e)
+            "unknown"
+        }
+
+        revisionEntity.request = request
+        revisionEntity.actor = actor
+        revisionEntity.traceId = traceId
     }
 }

src/main/kotlin/no/nav/klage/oppgave/util/TokenUtil.kt

@@ -218,4 +218,9 @@ class TokenUtil(
         tokenValidationContextHolder.getTokenValidationContext().getJwtToken(SecurityConfiguration.ISSUER_AAD)
             ?.jwtTokenClaims?.get("NAVident")?.toString()
             ?: throw RuntimeException("Ident not found in token")
+
+    fun getCallingApplication(): String =
+        tokenValidationContextHolder.getTokenValidationContext().getJwtToken(SecurityConfiguration.ISSUER_AAD)
+            ?.jwtTokenClaims?.get("azp_name")?.toString()
+            ?: throw RuntimeException("Application not found in token")
 }