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

SEC-1052: Add option to prevent URL rewriting of jsessionid #1303

Closed
spring-issuemaster opened this issue Dec 8, 2008 · 3 comments
Closed

SEC-1052: Add option to prevent URL rewriting of jsessionid #1303

spring-issuemaster opened this issue Dec 8, 2008 · 3 comments

Comments

@spring-issuemaster
Copy link

@spring-issuemaster spring-issuemaster commented Dec 8, 2008

Luke Taylor(Migrated from SEC-1052) said:

This is often seen as a security risk:

http://www.owasp.org/index.php/Top_10_2007-A7

“Do not expose any session identifiers or any portion of valid credentials in URLs or logs (no session rewriting or storing the user’s password in log files)”

We should add an option in the namespace (and the HttpSessionSecurityContextRepository) to override URL encoding and prevent the id being written to the URL.

@spring-issuemaster

This comment has been minimized.

Copy link
Author

@spring-issuemaster spring-issuemaster commented Dec 16, 2008

Luke Taylor said:

I’ve added the property disableUrlRewriting to HttpSessionSecurityContextRepository. Just need to add a corresponding one to the namespace.

@spring-issuemaster

This comment has been minimized.

Copy link
Author

@spring-issuemaster spring-issuemaster commented Dec 16, 2008

Luke Taylor said:

I’ve added support for “disable-url-rewriting” to the namespace parser.

@spring-issuemaster

This comment has been minimized.

Copy link
Author

@spring-issuemaster spring-issuemaster commented Feb 18, 2010

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants
You can’t perform that action at this time.