Issue Details (XML | Word | Printable)

Key: WICKET-1782
Type: Improvement Improvement
Status: Resolved Resolved
Resolution: Fixed
Priority: Major Major
Assignee: Igor Vaynberg
Reporter: Gorka Vicente
Votes: 0
Watchers: 2
Operations

If you were logged in you would be able to see more operations.
Wicket

Protection against CSRF (cross-site request forgery) attacks

Created: 08/Aug/08 11:26 AM   Updated: 22/Sep/08 05:50 AM
Return to search
Component/s: wicket
Affects Version/s: 1.3.4
Fix Version/s: 1.3.5, 1.4-RC1

Time Tracking:
Not Specified

Resolution Date: 08/Aug/08 10:14 PM


 Description  « Hide

Currently Wicket doesn't include a uniform and automatic solution against CRSF vulnerability or OWASP-A5 vulnerability [1].

In order to solve CSRF is necessary to avoid static HTML and create dynamic or aleatory HTML per user.

Two posible solutions:

1. Include a random token (aleatory parameter) to each url (link or form). The name and the value of this parameter can be the same per user or change per request (more secure but perform worse). It seems that can be implemented creating other implementation of IRequestCodingStrategy interface.

2. Encrypt all urls (links and form urls) using "Request Coding Strategy" strategy offered currently by wicket (CryptedUrlWebRequestCodingStrategy). Provide a security factory to use a different key per user or add some aleatory data to encrypted data (for example user jessionid). (SunJceCrypt, bundled in Wicket, is vulnerable to CSRF because obtained encrypted string is the same for all the users)


[1] http://www.owasp.org/index.php/Top_10_2007-A5



 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
No work has yet been logged on this issue.