[java-oidc-common] branch main updated: Only warn to logs if multiple values were found for Integer/Boolean.

[Henri Mikkonen]

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

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

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

The following commit(s) were added to refs/heads/main by this push:
       new  db977ba   Only warn to logs if multiple values were found for Integer/Boolean.
db977ba is described below

commit db977ba0cda3e9ba955422ad7522580574a53d91
Author: Henri Mikkonen <henri.mikkonen at iki.fi>
AuthorDate: Wed Mar 17 10:06:27 2021 +0200

    Only warn to logs if multiple values were found for Integer/Boolean.
---
 .../attribute/transcoding/impl/OIDCStringAttributeTranscoder.java     | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/oidc-common-attribute-impl/src/main/java/net/shibboleth/oidc/attribute/transcoding/impl/OIDCStringAttributeTranscoder.java b/oidc-common-attribute-impl/src/main/java/net/shibboleth/oidc/attribute/transcoding/impl/OIDCStringAttributeTranscoder.java
index 98a11e2..47d3fbd 100644
--- a/oidc-common-attribute-impl/src/main/java/net/shibboleth/oidc/attribute/transcoding/impl/OIDCStringAttributeTranscoder.java
+++ b/oidc-common-attribute-impl/src/main/java/net/shibboleth/oidc/attribute/transcoding/impl/OIDCStringAttributeTranscoder.java
@@ -123,7 +123,9 @@ public class OIDCStringAttributeTranscoder extends AbstractOIDCAttributeTranscod
         final StringBuilder attributeString = new StringBuilder();
         for (final Object value : values) {
             if (value instanceof Integer || value instanceof Boolean) {
-                log.warn("Values converted to Integer/Boolean but non-Array encoding, ignoring all but first value");
+                if (values.size() > 1) {
+                    log.warn("Values converted to Integer/Boolean but non-Array encoding, ignoring all but first");
+                }
                 return value;
             } else if (value instanceof String) {
                 if (!stringEmpty) {

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