Bug fix (Security Vulnerability)

[Adityakk9031]

What kind of change does this PR introduce?

Bug fix – security hardening for external links by ensuring safe usage of target="_blank" with rel="noopener noreferrer" in tooling components.

Issue Number

Closes #2227

Screenshots / Videos

N/A – this is a code-level security fix with no visual changes expected.

Summary

This PR addresses the security vulnerability described in #2227 by adding rel="noopener noreferrer" to all external links in ToolingTable.tsx and ToolingDetailModal.tsx. It also corrects incorrect usages of target="blank" to the standard target="_blank" so that external links open in a new tab while following security best practices.

Changes

• Added rel="noopener noreferrer" to external <a> tags rendered in ToolingTable.tsx.
• Added rel="noopener noreferrer" to external <a> tags rendered in ToolingDetailModal.tsx.
• Replaced all occurrences of target="blank" with the correct target="_blank" value on external links.

Does this PR introduce a breaking change?

No.

Documentation Updates

N/A.

Checklist

Please ensure the following tasks are completed before submitting this pull request.

• Read, understood, and followed the contributing guidelines.

[github-actions]

Hi @Adityakk9031! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description, completed checklist items

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[Adityakk9031]

ok i will fix the error later tomorrow

[github-actions]

Hi @Adityakk9031! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: kind of change description, completed checklist items

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[github-actions]

built with Refined Cloudflare Pages Action

⚡ Cloudflare Pages Deployment

Name	Status	Preview	Last Commit
website	✅ Ready (View Log)	Visit Preview	9c808e6

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (e242d60) to head (9c808e6).
⚠️ Report is 28 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff            @@
##              main     #2235   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines          660       663    +3     
  Branches       202       205    +3     
=========================================
+ Hits           660       663    +3     

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[github-actions]

Hi @Adityakk9031! Thanks a lot for your contribution!

I noticed that the following required information is missing or incomplete: completed checklist items

Please update the PR description to include this information. You can find placeholders in the PR template for these items.

Thanks a lot!

[Adityakk9031]

@jagpreetrahi hey please have a look

[Adityakk9031]

Hi @Utkarsh-123github, thanks for catching that!

In my previous commit, I accidentally removed the {bowtieData ? ... : } ternary logic in the mobile table section and replaced it with just the tag. This would have broken the fallback — tools without Bowtie data would render a broken link instead of the info icon.

I've fixed this in my latest commit. The changes are now minimal and purely security-focused:

Fixed target='blank' → target='_blank' (typo in original code)
Added rel='noopener noreferrer' to all external links
No structural or behavioral changes — the ternary fallback with InfoIcon is fully preserved
Could you please re-review? Thanks!

[jagpreetrahi]

lgtm

[Adityakk9031]

@jagpreetrahi please merge this pr and this one #2236

[Utkarsh-123github]

LGTM🚀
Thanks @Adityakk9031 for your efforts and contribution!