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

Commit 9fc5220

Browse files
author
Mario
committed
Merge branch '1158_handlers_fix_using_spring_security_in_front_stormpath' of github.com:stormpath/stormpath-sdk-java into 1123_spring_security_4.2.0_remove_apply
2 parents bd3e2a7 + 2433efc commit 9fc5220

File tree

4 files changed

+8
-8
lines changed

4 files changed

+8
-8
lines changed

extensions/servlet/src/main/java/com/stormpath/sdk/servlet/mvc/FormController.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -133,8 +133,8 @@ protected ViewModel doGet(HttpServletRequest request, HttpServletResponse respon
133133
protected Map<String,?> createModel(HttpServletRequest request, HttpServletResponse response) {
134134
List<ErrorModel> errors = null;
135135
// We need to look for error in queryString now that we moved SpringSecurity to in in front of us: https://github.com/stormpath/stormpath-sdk-java/issues/915
136-
String errorQueryString = request.getQueryString();
137-
boolean requestContainsError = request.getParameter("error") != null || (errorQueryString != null && errorQueryString.contains("error"));
136+
String queryString = request.getQueryString();
137+
boolean requestContainsError = request.getParameter("error") != null || (queryString != null && queryString.contains("error"));
138138
if (requestContainsError) {
139139
errors = new ArrayList<>();
140140
ErrorModel error = null;

extensions/spring/boot/stormpath-spring-security-webmvc-spring-boot-starter/src/main/java/com/stormpath/spring/boot/autoconfigure/StormpathWebSecurityAutoConfiguration.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public SocialCallbackSpringSecurityProcessingFilter socialCallbackSpringSecurity
135135
* @since 1.3.0
136136
*/
137137
@Bean
138-
@ConditionalOnMissingBean(name="contentNegotiationSpringSecurityAuthenticationFilter")
138+
@ConditionalOnMissingBean
139139
@Override
140140
public ContentNegotiationSpringSecurityAuthenticationFilter contentNegotiationSpringSecurityAuthenticationFilter() {
141141
return super.contentNegotiationSpringSecurityAuthenticationFilter();
@@ -161,8 +161,8 @@ public CorsConfigurationSource corsConfigurationSource() {
161161
*/
162162
@Bean
163163
@ConditionalOnMissingBean
164-
public AccountResolverFilter springSecuritResolvedAccountFilter() {
165-
return super.springSecuritResolvedAccountFilter();
164+
public AccountResolverFilter springSecurityResolvedAccountFilter() {
165+
return super.springSecurityResolvedAccountFilter();
166166
}
167167

168168
/**

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -256,7 +256,7 @@ public ContentNegotiationSpringSecurityAuthenticationFilter contentNegotiationSp
256256
/**
257257
* @since 1.3.0
258258
*/
259-
public AccountResolverFilter springSecuritResolvedAccountFilter() {
259+
public AccountResolverFilter springSecurityResolvedAccountFilter() {
260260
AccountResolverFilter accountResolverFilter = new AccountResolverFilter();
261261
accountResolverFilter.setEnabled(stormpathFilterEnabled);
262262
accountResolverFilter.setResolvers(stormpathAccountResolvers);

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -127,8 +127,8 @@ public CorsConfigurationSource corsConfigurationSource() {
127127
* @since 1.3.0
128128
*/
129129
@Bean
130-
public AccountResolverFilter springSecuritResolvedAccountFilter() {
131-
return super.springSecuritResolvedAccountFilter();
130+
public AccountResolverFilter springSecurityResolvedAccountFilter() {
131+
return super.springSecurityResolvedAccountFilter();
132132
}
133133

134134
/**

0 commit comments

Comments
 (0)