Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Spring GWTP-Dispatch secure action simplify configuration, by providing a default value for cookieName. #303

Merged
merged 1 commit into from
Aug 13, 2013

Conversation

imrabti
Copy link
Contributor

@imrabti imrabti commented Aug 10, 2013

Offer a default name for securityCookie when there any name defined on Spring scanned properties files.

The default name is JSESSIONID, which is the one used in most cases, otherwise the user need to override that by defining it on his property file, the key of the property is securityCookieName.

@christiangoudreau
Copy link
Member

Don't forget to document this (if it's not already done!)

@christiangoudreau
Copy link
Member

LGTM

branflake2267 added a commit that referenced this pull request Aug 13, 2013
Spring GWTP-Dispatch secure action simplify configuration, by providing a default value for cookieName.
@branflake2267 branflake2267 merged commit 3d79fae into master Aug 13, 2013
@imrabti
Copy link
Contributor Author

imrabti commented Aug 13, 2013

No documentation is required, it is more of a simplification, if the user forgot to specify the securityCookieName on server side the default name is used. No configuration is needed now to use secure action with Spring.

branflake2267 added a commit that referenced this pull request Apr 4, 2014
Spring GWTP-Dispatch secure action simplify configuration, by providing a default value for cookieName.
hpehl pushed a commit to hpehl/GWTP that referenced this pull request Dec 9, 2014
Spring GWTP-Dispatch secure action simplify configuration, by providing a default value for cookieName.


Former-commit-id: e7af56e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants