matrix-org/synapse

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

Aperta

#2860 aperta il 9 feb 2018

Vedi su GitHubGuida contributor
 (3 commenti) (0 reazioni) (0 assegnatari)Python (2196 fork)batch import
A-Email-PushP4T-EnhancementZ-Help-Wantedgood first issue

Metriche repository

Star
 (11.713 stelle)
Metriche merge PR
 (Nessuna PR mergiata in 30 g)

Descrizione

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

Guida contributor

Direzione di ricerca
Aggiorna la dipendenza Bleach nel file delle dipendenze del progetto, quindi modifica la funzione safe markup in synapse/push/mailer.py per utilizzare il nuovo parametro allowed protocols e decommentare le righe che limitano gli schemi di collegamento.
Tech stack
python
Dominio
backend
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
PythonGit
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70