Streamline email functionality
Ninguém assumiu esta issue ainda.
Avaliação
- Dificuldade
- 4/5
- Tempo estimado
- 3-5 dias
- Facilidade para iniciantes
- 45/100
Direção de pesquisa
Leia a função send_email de util.py e a implementação de e-mail em notifications.py; depois, rastreie os chamadores para notificações de recursos, e-mail de senha perdida e resumos de status. Compare a configuração e o comportamento em caso de falha descritos na issue; considera-se concluído quando todos os casos de envio de e-mail usam uma única função send_email robusta sem quebrar o comportamento de login opcional.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Python
- Estrelas
- 93
- Forks
- 77
- Merge médio
- 27min
- PRs com merge (30d)
- 1
Preparar o ambiente
- Inclui um Dockerfile ou arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de geopython/GeoHealthCheck
-
Data importAbertaquestion
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 38/100
geopython/GeoHealthCheck#522 · 1 comentário ·
-
FTP ProbeAbertaenhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
geopython/GeoHealthCheck#520 ·
-
bug
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 55/100
geopython/GeoHealthCheck#517 ·
-
enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 45/100
geopython/GeoHealthCheck#483 · 1 comentário ·
-
enhancement
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 35/100
geopython/GeoHealthCheck#472 ·
Todas as issues de geopython/GeoHealthCheck
Issues semelhantes
-
New InternshipAbertanew_internship
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
-
get_graph warns that the oneway column is missing when no way has a oneway tagTalvez já em andamento @HTenkanen assumiu hoje. Aberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 75/100
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
cython/cython#8110 · 2 comentários ·
Mantenedores costumam responder em até 1 dia
-
fix: security 🔐 style: article/section 📝 system: data-mgmt 🗃️ system: security 🔐 system: software 💻
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 65/100
uabrc/uabrc.github.io#1267 ·
-
Link Checker ReportAbertaBot Type: Bug Type: Documentation
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
Mantenedores costumam responder em até 1 dia