Skip to content

Fix broken external links and add security attributes#1345

Merged
viyatb merged 3 commits intoowtf:developfrom
SachinAditya:fix-external-links-security
Feb 15, 2026
Merged

Fix broken external links and add security attributes#1345
viyatb merged 3 commits intoowtf:developfrom
SachinAditya:fix-external-links-security

Conversation

@SachinAditya
Copy link
Copy Markdown
Contributor

Fixes #1336

Summary:

  • Added rel="noopener noreferrer" to external links in the website documentation
  • Standardized external links to open in a new tab
  • Fixed broken/incorrect external URLs in the help and troubleshooting sections

Security impact:

  • Prevents reverse tabnabbing attacks by blocking access to window.opener

@SachinAditya
Copy link
Copy Markdown
Contributor Author

Hi maintainers, just following up on this PR as it was opened 3 days ago.
Please let me know if any changes or additional information are needed from my side.
Thanks for your time!

@viyatb
Copy link
Copy Markdown
Member

viyatb commented Feb 15, 2026

can you remove the

freenode link? OWASP switched to Slack

Copy link
Copy Markdown
Member

@viyatb viyatb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove freenode and we can merge this

@SachinAditya
Copy link
Copy Markdown
Contributor Author

@viyatb
Thanks for feedback as per the feedback i removed the freenode link if any changes is still want to change then i am happen to do

@viyatb viyatb merged commit be16605 into owtf:develop Feb 15, 2026
1 check failed
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.

Broken external links and missing security attributes (rel="noopener noreferrer")

2 participants