Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

記事作成メール通知が届かない #99

Closed
irgaly opened this issue Oct 13, 2015 · 8 comments
Closed

記事作成メール通知が届かない #99

irgaly opened this issue Oct 13, 2015 · 8 comments

Comments

@irgaly
Copy link

irgaly commented Oct 13, 2015

Knowledge 0.6pre2へアップデート後、記事作成通知が届かなくなりました。すべての通知を受け取るように設定していますが、ユーザ追加通知のみ届き、他の通知は届かない状態となっています。

アップデート前はKnowledge0.5.3pre3でしたが、0.5.3pre3ではナレッジ登録やナレッジ更新通知が届いていました。

まずは起票いたします。

@koda-masaru
Copy link
Contributor

ご連絡ありがとうございます。

通知設定の部分で、マイグレーションが失敗しているのかもしれません。

または、メール通知処理にメールアドレスのフォーマットチェックを追加しましたので、そのチェックで、「メールアドレスでは無い」と判定されているかもしれません。

メールアドレスチェックは、正規表現で以下のパターンをチェックしています。
このフォーマットでメールアドレスと判定されますでしょうか?

^[a-zA-Z0-9!#$%&'_`/=~\\*\\+\\-\\?\\^\\{\\|\\}]+(\\.[a-zA-Z0-9!#$%&'_`/=~\\*\\+\\-\\?\\^\\{\\|\\}]+)*"
            + "@" + "[a-zA-Z0-9][a-zA-Z0-9\\-]*(\\.[a-zA-Z0-9\\-]+)*$

もしお時間があれば、ご確認願います。

@irgaly
Copy link
Author

irgaly commented Oct 15, 2015

Javaで上記正規表現をチェックしたところ、メールアドレスとして判定されました。

ユーザ追加通知が届き、記事作成通知が届かない点が気になっています。
ログやDBの中身など確認してみたいのですが、どの情報が必要でしょうか。

@nagodon
Copy link
Contributor

nagodon commented Oct 16, 2015

@koda-masaru @irgaly

こちらですがもしかすると記事が削除されたか何かで記事が取得できない時に
バッチ処理が落ちてしまって届いてないかもしれません。

こちらでもメールが送られない現象に遭遇していて調査していたのですが
記事やコメント、参考になったのデータが取得できない時にスキップする処理を入れてみたところ
通知処理が動くようになりました。

@irgaly

NotifyMailBat.logが下記の形でずっと流れてないでしょうか?

INFO  2015-10-17 00:13:29,747 [DBConnenctionLogic(35)] Custom connection setting is exists.
INFO  2015-10-17 00:14:29,300 [AbstractBat(26)] NotifyMailBat is start.
INFO  2015-10-17 00:14:29,664 [DBConnenctionLogic(35)] Custom connection setting is exists.
INFO  2015-10-17 00:15:31,811 [AbstractBat(26)] NotifyMailBat is start.

@nagodon
Copy link
Contributor

nagodon commented Oct 17, 2015

#106 こちらのPRでデータが取得できない場合はスキップするようにしてみました。

koda-masaru added a commit that referenced this issue Oct 18, 2015
#99 各通知キュー登録時にnullチェックをするように
@irgaly
Copy link
Author

irgaly commented Oct 19, 2015

ありがとうございます。NotifyMailBat.logのログはご指摘の通りとなっています。
#106 で指摘されているような、記事の作成 -> すぐに記事を削除 といった動作は通常の利用でもありえるため、同じ原因の可能性が高いと思います。

対応をマージしていただけましたので、次回リリース時に通知動作を確認してみます。

@koda-masaru
Copy link
Contributor

ありがとうございます。
週末あたりに、リリースしようと思いますので、お手数ですが確認をお願いいたします。

@koda-masaru koda-masaru self-assigned this Oct 19, 2015
@koda-masaru koda-masaru added this to the v0.6.0 milestone Oct 19, 2015
@koda-masaru koda-masaru reopened this Oct 19, 2015
@irgaly
Copy link
Author

irgaly commented Nov 10, 2015

遅くなりましたが、v0.6.1で動作を確認し、メールが送信できるようになっていることを確認しました。
ありがとうございました。:+1:

@koda-masaru
Copy link
Contributor

ご連絡ありがとうございます 😄
正しく動作したようですので安心いたしました。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants