Merge pull request #166 from Sachin-Mamoru/fix-logs-issue

Fixed issue of consider outcome for the callback
wso2-extensions · Jun 3, 2024 · 99836df · 99836df
2 parents b41ead7 + 5e0565c
commit 99836df
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/...n/java/org/wso2/carbon/identity/conditional/auth/functions/http/AbstractHTTPFunction.java b/...n/java/org/wso2/carbon/identity/conditional/auth/functions/http/AbstractHTTPFunction.java
@@ -174,6 +174,7 @@ protected void executeHttpMethod(HttpUriRequest clientRequest, Map<String, Objec
             if (!isRetry.shouldRetry()) {
                 return result;
             }
+            outcome = result.getRight().getLeft();
         }
         return Pair.of(isRetry, Pair.of(outcome, null));
     }

diff --git a/...wso2/carbon/identity/conditional/auth/functions/http/util/ClientCredentialAuthConfig.java b/...wso2/carbon/identity/conditional/auth/functions/http/util/ClientCredentialAuthConfig.java
@@ -292,7 +292,8 @@ private String attemptAccessTokenRequest(int maxAttempts) {
                                     attemptCount + ".")
                             .logDetailLevel(DiagnosticLog.LogDetailLevel.APPLICATION)
                             .resultStatus(DiagnosticLog.ResultStatus.FAILED);
-                    attemptCount++;         LoggerUtils.triggerDiagnosticLogEvent(diagnosticLogBuilder);
+                    attemptCount++;
+                    LoggerUtils.triggerDiagnosticLogEvent(diagnosticLogBuilder);
                 }
                 LOG.info("Retrying token request for session data key: " +
                         this.authenticationContext.getContextIdentifier() + ". Attempt: " + attemptCount);