-
Notifications
You must be signed in to change notification settings - Fork 680
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
Broken Links #292
Comments
Could you attach |
It's the contents of the code snippet in the OP. Would be great to use a better URL checker - the one I used didn't catch all the broken internal references, and its non-paid version only checked 2000 links out of, potentially, many more. |
Also, some of these links actually might resolve (just that there's a Javascript or 3xx code redirect), just one more peculiarity of the URL checker I used - in that case, IMO, the URL needs to be updated because, at some point, old links inevitably stop redirecting and start 404'ing. |
Thanks @CRImier |
This is also a broken one https://owasp.org/www-community/attacks/Tokenizing from the main page in CSRF. |
It seems there was never "Tokenizing" content: https://wiki.owasp.org/index.php/Tokenizing For that one it would be best to just remove the link. |
|
See OWASP#292 (comment) Signed-off-by: kingthorin <[email protected]>
See #292 (comment) Signed-off-by: kingthorin <[email protected]>
Page https://github.com/OWASP/www-community/blob/master/pages/Broken_Access_Control.md has this link: http://www.infosecuritymag.com/2002/jun/insecurity.shtml |
@fzipi let us know what your suggestion is, or just go ahead and open a PR. |
@kingthorin Added #393 with the links from the top 10. |
Would it be worth going through these and just linking to the wayback machine until suitable replacements can be identified? |
I’d suggest fixing a few properly (in batches or whatever), vs. having to go through them all twice. |
@CRImier can I work on this issue |
@draunger go for it. |
I concur, go for it^^ |
On page https://github.com/OWASP/www-community/blob/master/pages/Types_of_Cross-Site_Scripting.md there is "broken" (author of GitHub page has hidden the content) link in References [2] |
Was going through pages, found plenty of broken references. Ran a broken URL checker against the website, found a lot of broken URLs. Fixed some of them in #290 . Some URLs remain broken:
If anyone wants to go through these,
grep --color=always -nr -Ff broken_urls_left.txt|grep --color=always -v "broken_"|sort
will show where those URLs are specifically (might miss some of these, though). Could probably also find a lot of broken internal references by looking for"wikilink"
.The text was updated successfully, but these errors were encountered: