[java-opensaml COMMIT] in /trunk/opensaml-profile-api/src: main/java/org/opensaml/profile/context/ProfileRequestConte...

noreply at shibboleth.net noreply at shibboleth.net
Mon Jun 2 13:13:58 EDT 2014


Author: scantor
Date: Mon Jun  2 13:13:57 2014
New Revision: 3908

URL: http://svn.shibboleth.net/view/java-opensaml?rev=3908&view=rev
Log:
Default to non-browser profile flag, set correctly for browser profiles and propagate into authentication.

Modified:
    trunk/opensaml-profile-api/src/main/java/org/opensaml/profile/context/ProfileRequestContext.java
    trunk/opensaml-profile-api/src/test/java/org/opensaml/profile/context/ProfileRequestContextTest.java

Modified: trunk/opensaml-profile-api/src/main/java/org/opensaml/profile/context/ProfileRequestContext.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-profile-api/src/main/java/org/opensaml/profile/context/ProfileRequestContext.java?rev=3908&r1=3907&r2=3908&view=diff
==============================================================================
--- trunk/opensaml-profile-api/src/main/java/org/opensaml/profile/context/ProfileRequestContext.java (original)
+++ trunk/opensaml-profile-api/src/main/java/org/opensaml/profile/context/ProfileRequestContext.java Mon Jun  2 13:13:57 2014
@@ -51,7 +51,6 @@
     /** Constructor. */
     public ProfileRequestContext() {
         profileId = ANONYMOUS_PROFILE_ID;
-        browserProfile = true;
     }
 
     /**
@@ -78,7 +77,7 @@
     }
 
     /**
-     * Get whether the current profile request is browser-based (defaults to true).
+     * Get whether the current profile request is browser-based (defaults to false).
      * 
      * @return whether the current profile request is browser-based
      */

Modified: trunk/opensaml-profile-api/src/test/java/org/opensaml/profile/context/ProfileRequestContextTest.java
URL: http://svn.shibboleth.net/view/java-opensaml/trunk/opensaml-profile-api/src/test/java/org/opensaml/profile/context/ProfileRequestContextTest.java?rev=3908&r1=3907&r2=3908&view=diff
==============================================================================
--- trunk/opensaml-profile-api/src/test/java/org/opensaml/profile/context/ProfileRequestContextTest.java (original)
+++ trunk/opensaml-profile-api/src/test/java/org/opensaml/profile/context/ProfileRequestContextTest.java Mon Jun  2 13:13:57 2014
@@ -26,10 +26,10 @@
 
     @Test public void testBrowserProfile() {
         final ProfileRequestContext context = new ProfileRequestContext();
+        Assert.assertFalse(context.isBrowserProfile());
+
+        context.setBrowserProfile(true);
         Assert.assertTrue(context.isBrowserProfile());
-
-        context.setBrowserProfile(false);
-        Assert.assertFalse(context.isBrowserProfile());
     }
 
     @Test public void testProfileId() {



More information about the commits mailing list