jsx-eslint/eslint-plugin-react

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

Open

#2.398 aberto em 2 de set. de 2019

Ver no GitHub
 (7 comments) (1 reaction) (0 assignees)JavaScript (2.797 forks)batch import
enhancementhelp wanted

Métricas do repositório

Stars
 (8.630 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Revise a implementação da regra jsx no target blank em lib/rules/jsx no target blank.js. Examine a lógica existente que verifica os atributos rel e considere como o atributo referrerPolicy deve ser tratado como equivalente a rel='noopener noreferrer'. Verifique o arquivo de teste para casos de teste existentes e adicione novos. Consulte os comentários na issue para qualquer abordagem sugerida.
Pilha de tecnologia
javascriptreact
Domain
frontend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
JavaScriptESLint
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70