daemonfreaks.com

trunkのcomments plugin

posted by jun-g at Tue, 12 Dec 2006 03:00 JST

先日のtrackback pluginのパッチは無事に取り込まれた模様。よかった。

そのパッチを作る際、trunkのcomments.pyを軽く見たんやけど、なんかメール送信時の非ASCII文字を受け付けない問題とか色々修正されてるっぽかったので、近々試してみないとなぁと思ってたら、shunuhsさんがtrunkに入れ替えた上にパッチを書いて、しかもtb spam用のプラグインまで作ってた。素晴らしいっす。仕事早いっす。動作実績があると心強いので、早速僕もtrunkのcomments/trackback pluginに入れ替える事にした。最近コメントスパムに悩まされていたので、ついでにMagicWord pluginも導入することに。

が、その前にshunuhsさんが「メールが飛ばない」と言われている問題を調べてみる事にした。テスト環境のcomments.pyをtrunkのものに入れ替えて試してみると、pyblosxomのエラーログに

2006-12-12 00:07:34,909 [ERROR] comments: error sending email: ['Traceback (most recent call last):\n', '  File "/pub/www/pyblosxom/plugins/comments.py", line 524, in send_email\n    msg=body)\n', '  File "/usr/local/lib/python2.4/smtplib.py", line 692, in sendmail\n    (code,resp) = self.data(msg)\n', '  File "/usr/local/lib/python2.4/smtplib.py", line 485, in data\n    q = quotedata(msg)\n', '  File "/usr/local/lib/python2.4/smtplib.py", line 193, in quotedata\n    re.sub(r\'(?:\\r\\n|\\n|\\r(?!\\n))\', CRLF, data))\n', '  File "/usr/local/lib/python2.4/sre.py", line 142, in sub\n    return _compile(pattern, 0).sub(repl, string, count)\n', 'TypeError: expected string or buffer\n']

というのが出てた。どうもメール送信時にエラーになってるっぽい。で、なんとなくピンときて入れた修正が以下。

--- comments.py.orig	Tue Dec 12 01:52:47 2006
+++ comments.py	Tue Dec 12 02:36:47 2006
@@ -521,7 +521,7 @@
             server = smtplib.SMTP(config['comment_smtp_server'])
             server.sendmail(from_addr=email,
                             to_addrs=config['comment_smtp_to'], 
-                            msg=body)
+                            msg=body.as_string())
             server.quit()
 
     except Exception, e:

予想適中、無事にメール送信できるようになった。しかーし!現状のcomments.pyの作りだと、メールヘッダに設定されるべき「From」「To」「Subject」がメール本文に記載されるという超手抜き仕様。現状だとこんな本文のメールが来る。

From: from_at_aemonfreaks.com
To: to_at_daemonfreaks.com
Date: Mon, 11 Dec 2006 16:20:32 -0000
Subject: comment by TEST

Name: TEST
URL: 
Hostname: localhost.daemonfreaks.local (192.168.0.21)
Entry URL: http://localhost/blog/200611060000
Comment location: /pub/www/pyblosxom/entries/comments/200611060000-1165854032.63.cmt


Hello, world.

しかも件名が空。イケてない。先頭の4行は明らかにメールヘッダにあるべき。なので、以前自分で修正したやつと同様の修正を今回も入れる事にした。修正内容は以下のとおり。

--- comments.py.orig	Tue Dec 12 01:52:47 2006
+++ comments.py	Tue Dec 12 01:48:48 2006
@@ -475,13 +475,9 @@
         comment_dir = os.path.join(config['comment_dir'], entry['absolute_path'])
 
         # create the message
+        from email.Header import Header
         from email.MIMEText import MIMEText
         message = []
-        message.append("From: %s" % email)
-        message.append("To: %s" % config["comment_smtp_to"])
-        message.append("Date: %s" % formatdate(float(comment['pubDate'])))
-        message.append("Subject: comment by %s" % author)
-        message.append("")
         message.append("Name: %s" % author)
         if comment.has_key('email'):
             message.append("Email: %s" % comment['email'])
@@ -498,7 +494,13 @@
 
         body = '\n'.join(message)
         body = MIMEText(body.encode('utf-8'), 'plain', 'utf-8')
- 
+        subj = "comment by %s" % author
+        subj = Header(subj.encode("utf-8"), "utf-8")
+        body["Subject"] = subj
+        body["From"] = config["comment_smtp_from"]
+        body["To"] = config["comment_smtp_to"]
+        body["Date"] = formatdate(float(comment['pubDate']))
+
         if (config.has_key('comment_mta_cmd')):
             argv = [config['comment_mta_cmd'],
                     '-s',
@@ -506,7 +508,7 @@
                     config['comment_smtp_to']]
             # TODO: switch to subprocess when we can require python 2.4
             process = popen2.Popen3(argv, capturestderr=True)
-            process.tochild.write(body)
+            process.tochild.write(body.as_string())
             process.tochild.close()
             process.wait()
             stdout = process.fromchild.read()
@@ -521,7 +523,7 @@
             server = smtplib.SMTP(config['comment_smtp_server'])
             server.sendmail(from_addr=email,
                             to_addrs=config['comment_smtp_to'], 
-                            msg=body)
+                            msg=body.as_string())
             server.quit()
 
     except Exception, e:

これで「Subject」「From」「To」(「Date」はオマケ)が正しくメールヘッダに設定されるようになった。ちなみに「comment_mta_cmd」を使用してメール送信を行う場合の動作は未確認なので、どなたか試してみてください…。

と、ここまで調べたところで時間切れになってしまった。続きは明日以降で。Magic Wordとtb_spam.pyの動作確認をしてから全部導入することにしよう。

posted by shunuhs at Thu Dec 14 01:23:10 2006

素晴らしい!ありがとうございました!

結構な修正量だったので、とりあえず一番最初にあったbody.as_string()の修正だけ、入れてみました。
ちゃんとメール飛んでますが確かにめちゃめちゃ手抜きなメールですね(笑)

この修正もtrunkにpatchを取り込んでもらえるといいですね。
私がMLに送ったtrackback.pyへの修正要望は残念ながら放置気味ですが・・

tb_spam.pyは超暫定対応でエラー処理も何も入れてませんがとりあえず私のところでは元気にtrackback spamをはじいてくれています。

posted by jun-g at Fri Dec 15 03:56:54 2006

いえいえ、こちらこそtb_spam.pyをありがたく利用させて頂きます!

trackback.pyとmagicword.pyへのパッチですが、MagicWordプラグインはcontribパッケージに含まれていない(SVNで管理されていない)ので、MagicWordプラグインの利用を前提にしたtrackbackプラグインへの修正はもしかしたら取り込んでもらえないのかも、と思いました。

その意思表示かどうかはわかりませんが、shunuhsさんのパッチがMLに流れた2日後に、新しいコメントスパム対策用のプラグインがSVNに登録されていました。
http://www.daemonfreaks.com/blog/200612150305.html

っていうか、「パッチ取り込んだよ」とか「これは取り込めないよ」とかMLに返信ぐらいしてくれてもいいのに…。


comments.pyのパッチは週末にでもMLに投げてみます。

Comments / TrackBacks

TrackBack URL for this entry: https://www.daemonfreaks.com/blog/trackback/200612120210








コメントスパム対策の為、各入力項目へのURL記載は全てリジェクトしてます。ごめんなさい。