dpkp/kafka-python

Only log warning about old SSL module when actually using SSL

Chiusa

#925 aperta il 30 dic 2016

Vedi su GitHub
 (0 commenti) (1 reazione) (0 assegnatari)Python (1475 fork)batch import
help wanted

Metriche repository

Star
 (5900 stelle)
Metriche merge PR
 (Merge medio 8h 49m) (75 PR mergiate in 30 g)

Descrizione

related to #924

Personally, I'd prefer this warning only be logged if we're actually trying to use SSL. If we're just using PLAINTEXT connections, there's no need to warn about this.

Guida contributor

Direzione di ricerca
L'avviso è probabilmente registrato nel codice relativo a SSL (ad es. in `kafka/conn.py` o `kafka/protocol/ssl.py`). Il contributore dovrebbe trovare la chiamata `warn` o `logger.warning` che menziona 'old SSL module' e racchiuderla in un'istruzione `if` che verifichi se la connessione utilizza SSL (ad es. `if self.config.get('security protocol') != 'PLAINTEXT'`). L'issue correlata #924 potrebbe fornire dettagli aggiuntivi sulla deprecazione del modulo SSL.
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.
Meno di 1 ora
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Nessuno
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75