matrix-org/synapse

Upgrade to Bleach 1.5, to limit link schemes to an allowlist

Aberta

#2.860 aberto em 9 de fev. de 2018

Ver no GitHubGuia do colaborador
 (3 comentários) (0 reação) (0 responsável)Python (2.196 forks)batch import
A-Email-PushP4T-EnhancementZ-Help-Wantedgood first issue

Métricas do repositório

Stars
 (11.713 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

The 1.5 version of Bleach made the allowed protocols configurable: http://bleach.readthedocs.io/en/latest/clean.html#allowed-protocols-protocols

Once the dependency is updated, the safe_markup function in the mailer can be updated and these lines can be uncommented: https://github.com/matrix-org/synapse/blob/42b50483be2b022735f8ae2107314d51e92e8471/synapse/push/mailer.py#L76-L77

Guia do colaborador

Direção de pesquisa
Atualize a dependência do Bleach no arquivo de dependências do projeto, depois modifique a função safe markup em synapse/push/mailer.py para usar o novo parâmetro allowed protocols e descomente as linhas que limitam os esquemas de link.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Funcionalidade
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.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
PythonGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70