dpkp/kafka-python

Only log warning about old SSL module when actually using SSL

Geschlossen

#925 geöffnet am 30.12.2016

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)Python (1.475 Forks)batch import
help wanted

Repository-Metriken

Stars
 (5.900 Sterne)
PR-Merge-Metriken
 (Durchschn. Merge 8h 49m) (75 gemergte PRs in 30 T)

Beschreibung

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.

Contributor Guide

Research-Richtung
Die Warnung wird wahrscheinlich im SSL bezogenen Code protokolliert (z. B. in `kafka/conn.py` oder `kafka/protocol/ssl.py`). Der Mitwirkende sollte den `warn` oder `logger.warning` Aufruf finden, der 'old SSL module' erwähnt, und ihn in eine `if` Anweisung einbetten, die prüft, ob die Verbindung SSL verwendet (z. B. `if self.config.get('security protocol') != 'PLAINTEXT'`). Das verwandte Issue #924 könnte zusätzliche Details zur Abschaffung des SSL Moduls liefern.
Tech Stack
python
Domain
backend
Issue Type
Funktion
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.
Unter 1 Stunde
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Veraltet
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Keine
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
75