Skip to content
This repository has been archived by the owner on Nov 20, 2024. It is now read-only.

noopener noreferrer added to external link to stop the target page ac… #1034

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

bobby-didcoding
Copy link
Contributor

…cessing the DOM

CodeQl : HTML links that open in a new tab or window allow the target page to access the DOM of the origin page using window.opener unless link type noopener or noreferrer is specified. This is a potential security risk.

Workflow

Merging

  • This PR can be merged by reviewers.

@bobby-didcoding bobby-didcoding requested a review from a team as a code owner April 25, 2024 14:28
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.74%. Comparing base (7eec0b6) to head (3b23fa9).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1034   +/-   ##
========================================
  Coverage    98.74%   98.74%           
========================================
  Files           42       42           
  Lines         1908     1908           
========================================
  Hits          1884     1884           
  Misses          24       24           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants