-
Notifications
You must be signed in to change notification settings - Fork 47
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
Updated documentation - Open Issue #272
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello @swetak20,
Thank you for taking the time to contribute to the project, but unfortunately changing the URL is not enough. Please read the ticket. All the content from https://wiki.owasp.org/index.php/CSRFGuard_3_Token_Injection should be moved to this GitHub repository (correctly formatted), so it would show on the https://owasp.org/www-project-csrfguard/ page.
I was working on migrating the contents to https://owasp.org/www-project-csrfguard/ and I believe it is currently generated from index.md. Could you confirm if this is correct? |
I was checking the link in https://wiki.owasp.org/index.php/CSRFGuard_3_Token_Injection before migrating them, and they redirect to a 404. Could you suggest what should we cite the sources as in the updated documentation? |
Yes, the documentation is generated from the markdown files. |
The test properties files can be found in the following folder in the repository: https://github.com/OWASP/www-project-csrfguard/tree/master/csrfguard-test/csrfguard-test-jsp/src/main/webapp/WEB-INF/classes |
@forgedhallpass I've made the requested changes. Please review the updates and let me know if there are any further adjustments needed. |
Updated Documentation as per Issue #36