[java-idp-oidc] branch main updated: JOIDC-112 - PROTOCOL_MESSAGE.OAUTH2 log appender to trace all the exchanged protocol messages

Henri Mikkonen henri.mikkonen at iki.fi
Tue Jun 28 13:58:28 UTC 2022


This is an automated email from the git hooks/post-receive script.

hjmikkon pushed a commit to branch main
in repository java-idp-oidc.

View the commit online:
http://git.shibboleth.net/view/?p=java-idp-oidc.git;a=commit;h=f15a0a4aff2ca01e05b190c856e7aef0adc2348f

The following commit(s) were added to refs/heads/main by this push:
     new f15a0a4a JOIDC-112 - PROTOCOL_MESSAGE.OAUTH2 log appender to trace all the exchanged protocol messages
f15a0a4a is described below

commit f15a0a4aff2ca01e05b190c856e7aef0adc2348f
Author: Henri Mikkonen <henri.mikkonen at iki.fi>
AuthorDate: Tue Jun 28 16:57:23 2022 +0300

    JOIDC-112 - PROTOCOL_MESSAGE.OAUTH2 log appender to trace all the exchanged protocol messages
    
    https://shibboleth.atlassian.net/browse/JOIDC-112
    
    Included omitNullValues() where it was forgotten.
---
 .../shibboleth/idp/plugin/oidc/op/decoding/impl/RequestUtil.java    | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/idp-oidc-extension-impl/src/main/java/net/shibboleth/idp/plugin/oidc/op/decoding/impl/RequestUtil.java b/idp-oidc-extension-impl/src/main/java/net/shibboleth/idp/plugin/oidc/op/decoding/impl/RequestUtil.java
index fcd63487..d83996ea 100644
--- a/idp-oidc-extension-impl/src/main/java/net/shibboleth/idp/plugin/oidc/op/decoding/impl/RequestUtil.java
+++ b/idp-oidc-extension-impl/src/main/java/net/shibboleth/idp/plugin/oidc/op/decoding/impl/RequestUtil.java
@@ -77,7 +77,7 @@ public final class RequestUtil {
      * @return The log message
      */
     @Nullable public static String getClientAuthenticationLog(@Nullable final ClientAuthentication authentication) {
-        return authentication == null ? null : MoreObjects.toStringHelper("ClientAuthentication")
+        return authentication == null ? null : MoreObjects.toStringHelper("ClientAuthentication").omitNullValues()
                 .add("clientId", authentication.getClientID())
                 .add("method", authentication.getMethod())
                 .toString();
@@ -90,7 +90,7 @@ public final class RequestUtil {
      * @return The log message
      */
     @Nullable public static String getAccessTokenLog(@Nullable final AccessToken accessToken) {
-        return accessToken == null ? null : MoreObjects.toStringHelper("AccessToken")
+        return accessToken == null ? null : MoreObjects.toStringHelper("AccessToken").omitNullValues()
                 .add("lifetime", accessToken.getLifetime())
                 .add("issuedTokenType", accessToken.getIssuedTokenType())
                 .add("parameterNames", accessToken.getParameterNames())
@@ -143,7 +143,7 @@ public final class RequestUtil {
      * @return The log message
      */
     @Nullable public static String getRefreshTokenLog(@Nullable final RefreshToken refreshToken) {
-        return refreshToken == null ? null : MoreObjects.toStringHelper("RefreshToken")
+        return refreshToken == null ? null : MoreObjects.toStringHelper("RefreshToken").omitNullValues()
                 .add("parameterNames", refreshToken.getParameterNames())
                 .add("value", refreshToken.getValue())
                 .toString();

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list