containers / containers/podman.io
Links with target="_blank" are missing rel="noopener noreferrer"
- Dominant language
- TypeScript
- Stars
- 108
- Forks
- 188
- Avg merge
- 9d 15h
- Merged PRs (30d)
- 11
Description
## What happened
- `src/components/content/ThankYouSection/index.tsx` — 7 sponsor logo links
- `src/components/ui/ArticleCard/index.tsx` — 2 article title links
- `src/pages/features.tsx` (~line 214) — the "on our Blog!" link
- `src/components/ui/Testimonial/index.tsx` (~line 34) — the featured link
Without `rel="noopener"`, a page opened via `target="_blank"` can access `window.opener` and navigate the original tab elsewhere (reverse tabnabbing); `noreferrer` additionally avoids leaking the referrer. This is exactly what the `react/jsx-no-target-blank` ESLint rule guards against. While current major browsers imply `noopener` for `target="_blank"`, setting it explicitly restores protection for older browsers and is the widely recommended best practice.
## What you expected to happen
External links that open in a new tab should include `rel="noopener noreferrer"`.
## Suggested fix
Add `rel="noopener noreferrer"` to each `target="_blank"` anchor, e.g.:
```tsx
```
I'd be happy to open a PR covering all 11 links.
Contributor guide
No contributing guide indexed for this repository
Research direction
Inspect the 11 target="_blank" links in src/components/content/ThankYouSection/index.tsx, src/components/ui/ArticleCard/index.tsx, src/pages/features.tsx, and src/components/ui/Testimonial/index.tsx. Add the requested rel value to each anchor, then search the site for remaining target="_blank" links and confirm the react/jsx-no-target-blank rule is satisfied.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- typescript
- Domain
- frontend, security
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Clearly specified
- Newbie friendliness
- 84/100