WEB-827: Prevent tabnabbing in window.open#3341
Conversation
Add noopener,noreferrer when opening external links to prevent the opened page from accessing window.opener (tabnabbing protection). Made-with: Cursor
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Note
|
Description:
External links are opened using window.open(url, '_blank'). Without noopener or noreferrer, the opened page can access window.opener, creating a potential tabnabbing vulnerability.
Fix:
Update window.open calls to include 'noopener,noreferrer'
ticket -> https://mifosforge.jira.com/browse/WEB-827?atlOrigin=eyJpIjoiNjM1YmU1YjExZDlmNDk3NWJlNDhlMGY1YTRjODMzYmEiLCJwIjoiaiJ9