Streamline email functionality
まだ誰も着手していません。
評価
調査の方向性
util.py の send_email 関数と notifications.py のメール実装を読み、その後、リソース通知、パスワード紛失メール、ステータス概要に関する呼び出し元を追跡します。issue に記載された設定と失敗時の動作を比較します。完了の条件は、メール送信のすべてのケースが、オプションのログイン動作を壊すことなく、1 つの堅牢な send_email 関数を使用することです。
索引モデルが issue の本文から書いたものです。
説明
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.
- 主要言語
- Python
- スター
- 92
- フォーク
- 76
- 平均マージ
- 7時間 21分
- マージ済み PR（30日）
- 3
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
geopython/GeoHealthCheck のほかの issue
-
Data importオープンquestion
難易度 4/5 3〜5日 初心者へのやさしさ 38/100
geopython/GeoHealthCheck#522 · コメント 1 件 ·
-
FTP Probeオープンenhancement
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
geopython/GeoHealthCheck#520 ·
-
bug
難易度 3/5 1〜2日 初心者へのやさしさ 55/100
geopython/GeoHealthCheck#517 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 45/100
geopython/GeoHealthCheck#483 · コメント 1 件 ·
-
enhancement
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
geopython/GeoHealthCheck#472 ·
geopython/GeoHealthCheck の issue をすべて見る
似ている issue
-
area: desktop area: website priority: P2 type: feature
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
appandflow/stim#3411 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1時間未満 初心者へのやさしさ 88/100
baptistehamon/lsapy#185 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
PolicyEngine/policyengine-us#10073 ·
メンテナーはふだん 2 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
syedhamidali/radarx#277 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
Arekkazu/sgpmp-backend#549 ·
メンテナーはふだん 1 日以内に返信