Streamline email functionality
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 45/100
Piste de recherche
Lisez la fonction send_email de util.py et l’implémentation des e-mails dans notifications.py, puis suivez leurs appelants pour les notifications de ressources, les e-mails de mot de passe oublié et les récapitulatifs d’état. Comparez la configuration et le comportement en cas d’échec décrits dans l’issue ; le travail est terminé lorsque tous les cas d’envoi d’e-mails utilisent une seule fonction send_email robuste sans casser le comportement de connexion facultative.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Is your feature request related to a problem? Please describe.
Email is sent in two places: in notifications.py and in util.py (send_email function).
Via #416 some fixes were made in notifications.py to cope with various email configurations TLS, no Auth etc, for both Docker and regular installs. This is for Resource notification only. All other email (lost password, status summary, ..) is still sent
with util.py-send_email.
Describe the solution you'd like
Provide a single send_email() function for all email-sending cases. Best is to take util.py-send_email and extend with the implementation in notifications.py. Make the function robust for config errors. May still be lean/silently fail, on login() failure as that is optional and may break existing installs.
Describe alternatives you've considered
We could discard SMTPLib and use a higher level email lib. But that also introduces an additional dependency, plus may break existing installations.
Additional context
N.A.
- Langage dominant
- Python
- Étoiles
- 92
- Forks
- 77
- Merge moyen
- 1 h 39 min
- PR mergées (30 j)
- 3
Préparer son environnement
- Fournit un Dockerfile ou un fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de geopython/GeoHealthCheck
-
Data importOuvertequestion
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
geopython/GeoHealthCheck#522 · 1 commentaire ·
-
FTP ProbeOuverteenhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
geopython/GeoHealthCheck#520 ·
-
bug
Difficulté 3/5 1-2 jours Accessibilité débutants 55/100
geopython/GeoHealthCheck#517 ·
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 45/100
geopython/GeoHealthCheck#483 · 1 commentaire ·
-
enhancement
Difficulté 4/5 3-5 jours Accessibilité débutants 35/100
geopython/GeoHealthCheck#472 ·
Toutes les issues de geopython/GeoHealthCheck
Issues similaires
-
The changelog workflow excludes otelbot but not dependabot, so every Dependabot pull request fails the changelog checkPeut-être pris @ocelotl l’a pris aujourd’hui. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 82/100
open-telemetry/opentelemetry-python#5825 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
cuga-project/cuga-agent#1020 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 1/5 Moins d'une heure Accessibilité débutants 70/100
microsoft/playwright#43640 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
stac-utils/stac-fastapi-elasticsearch-opensearch#965 ·
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour