jsx-eslint/eslint-plugin-react

Don't warn about react/jsx-no-target-blank when referrerPolicy is used

Open

#2.398 geöffnet am 2. Sept. 2019

Auf GitHub ansehen
 (7 Kommentare) (1 Reaktion) (0 zugewiesene Personen)JavaScript (2.797 Forks)batch import
enhancementhelp wanted

Repository-Metriken

Stars
 (8.630 Stars)
PR-Merge-Metriken
 (Keine gemergten PRs in 30 T)

Beschreibung

We have in our code:

<a rel="noopener" referrerPolicy="no-referrer" target="_blank" href={fooURL}>foo</a>

But react/jsx-no-target-blank is complaining about it with:

 Using target="_blank" without rel="noopener noreferrer" is a security risk: see https://mathiasbynens.github.io/rel-noopener  react/jsx-no-target-blank

Isn't it the same to have rel="noopener" and referrerPolicy="no-referrer" as having rel="noopener noreferrer"?

eslint-plugin-react@7.14.3

Contributor Guide

Research-Richtung
Überprüfe die Implementierung der jsx no target blank Regel in lib/rules/jsx no target blank.js. Untersuche die bestehende Logik, die rel Attribute prüft, und überlege, wie das referrerPolicy Attribut als äquivalent zu rel='noopener noreferrer' behandelt werden sollte. Überprüfe die Testdatei auf vorhandene Testfälle und füge neue hinzu. Sieh dir die Kommentare im Issue für etwaige vorgeschlagene Ansätze an.
Tech Stack
javascriptreact
Domain
frontend
Issue Type
Fehler
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 Stunden
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
JavaScriptESLint
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70