dpkp/kafka-python

Only log warning about old SSL module when actually using SSL

Fermée

#925 ouverte le 30 déc. 2016

Voir sur GitHub
 (0 commentaire) (1 réaction) (0 personne assignée)Python (1 475 forks)batch import
help wanted

Métriques du dépôt

Stars
 (5 900 étoiles)
Métriques de merge PR
 (Merge moyen 8h 49m) (75 PRs mergées en 30 j)

Description

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.

Guide contributeur

Direction de recherche
L'avertissement est probablement enregistré dans le code lié au SSL (par exemple dans `kafka/conn.py` ou `kafka/protocol/ssl.py`). Le contributeur doit trouver l'appel `warn` ou `logger.warning` qui mentionne 'old SSL module' et l'encadrer dans une instruction `if` qui vérifie si la connexion utilise SSL (par exemple `if self.config.get('security protocol') != 'PLAINTEXT'`). L'issue associée #924 pourrait fournir des détails supplémentaires sur la dépréciation du module SSL.
Stack technique
python
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Moins d'une heure
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Aucun
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
75