Skip to content
This repository was archived by the owner on Dec 12, 2018. It is now read-only.

Commit 937cf57

Browse files
committed
Updated default for id site callback to be per framework spec
1 parent 77da61b commit 937cf57

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

extensions/spring/stormpath-spring-security-webmvc/src/main/java/com/stormpath/spring/config/StormpathSecurityConfigurerAdapter.java

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -153,9 +153,13 @@ public class StormpathSecurityConfigurerAdapter extends AbstractStormpathSecurit
153153
@Value("#{ @environment['stormpath.web.callback.enabled'] ?: true }")
154154
protected boolean callbackEnabled;
155155

156-
@Value("#{ @environment['stormpath.web.idSite.resultUri'] ?: '/idSiteResult' }")
156+
// both idSiteResultUri and samlResultUri default to `/stormpathCallback`
157+
// this is a fix for https://github.com/stormpath/stormpath-sdk-java/issues/1254
158+
// TODO - for 2.0.0 release remove stormpath.web.idSite.resultUri and use stormpath.web.callback.uri for both id site and saml
159+
@Value("#{ @environment['stormpath.web.idSite.resultUri'] ?: '/stormpathCallback' }")
157160
protected String idSiteResultUri;
158161

162+
// TODO - for 2.0.0 release rename variable webCallbackUri
159163
@Value("#{ @environment['stormpath.web.callback.uri'] ?: '/stormpathCallback' }")
160164
protected String samlResultUri;
161165

0 commit comments

Comments
 (0)