Back to all reviewers

Secure external links

langfuse/langfuse
Based on 3 comments
TSX

Always add `rel="noopener noreferrer"` to external links that use `target="_blank"` to prevent tabnabbing attacks. This security attribute prevents malicious websites from gaining access to your window object through the opener property, which could be exploited for phishing attacks or other security breaches.

Security TSX

Reviewer Prompt

Always add rel="noopener noreferrer" to external links that use target="_blank" to prevent tabnabbing attacks. This security attribute prevents malicious websites from gaining access to your window object through the opener property, which could be exploited for phishing attacks or other security breaches.

Example:

// Insecure: vulnerable to tabnabbing
<Link 
  href="https://langfuse.com/docs/analytics/posthog"
  target="_blank"
>
  Integration Docs โ†—
</Link>

// Secure: protected against tabnabbing
<Link 
  href="https://langfuse.com/docs/analytics/posthog"
  target="_blank" 
  rel="noopener noreferrer"
>
  Integration Docs โ†—
</Link>

This security measure should be applied consistently across all external links in the application, especially in components that render user-provided or dynamic URLs.

3
Comments Analyzed
TSX
Primary Language
Security
Category

Source Discussions