[java-opensaml] 06/11: Switch to TreeMap for context builder so toString() sorted by keys.

Brent Putman putmanb at georgetown.edu
Thu Jan 30 02:17:31 EST 2020


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

putmanb pushed a commit to branch master
in repository java-opensaml.

View the commit online:
http://git.shibboleth.net/view/?p=java-opensaml.git;a=commit;h=46cdae4c100f722d2e84d30272dad912cc9c2a40

commit 46cdae4c100f722d2e84d30272dad912cc9c2a40
Author: Brent Putman <putmanb at georgetown.edu>
AuthorDate: Wed Jan 29 21:11:40 2020 -0500

    Switch to TreeMap for context builder so toString() sorted by keys.
    
    Makes debugging and visual inspection easier.
---
 .../saml2/profile/impl/DefaultAssertionValidationContextBuilder.java  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/profile/impl/DefaultAssertionValidationContextBuilder.java b/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/profile/impl/DefaultAssertionValidationContextBuilder.java
index 96c11be..2e006e8 100644
--- a/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/profile/impl/DefaultAssertionValidationContextBuilder.java
+++ b/opensaml-saml-impl/src/main/java/org/opensaml/saml/saml2/profile/impl/DefaultAssertionValidationContextBuilder.java
@@ -24,9 +24,9 @@ import java.security.cert.X509Certificate;
 import java.time.Duration;
 import java.util.Arrays;
 import java.util.Collections;
-import java.util.HashMap;
 import java.util.Map;
 import java.util.Set;
+import java.util.TreeMap;
 import java.util.function.Function;
 import java.util.function.Predicate;
 
@@ -491,7 +491,7 @@ public class DefaultAssertionValidationContextBuilder
     @Nonnull protected Map<String,Object> buildStaticParameters(
             @Nonnull final AssertionValidationInput input) {
         
-        final HashMap<String, Object> staticParams = new HashMap<>();
+        final TreeMap<String, Object> staticParams = new TreeMap<>();
         
         //For signature validation
         staticParams.put(SAML2AssertionValidationParameters.SIGNATURE_REQUIRED, 

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


More information about the commits mailing list