tDiary

はじめまして，munedaと申します．

On Wed, 27 Dec 2006 11:57:45 +0900
Kazuhiko <kazuhiko@...> wrote:

> かずひこです。
> 
> my-exプラグインを有効にした上で、wikiスタイルを使うと、
> 
> 2006-12-14 zunda <zunda at freeshell.org>
>         * my-ex.rb: allow a URL as the first argument
> 
> の変更で、[[hoge|20060102#p01]] という形式でのリンクが機能しなく
> なったので、原因を調べてみました。
> 
> wiki_style.rb では、
>   [[hoge|20060102#p01]]
> を
>   <%=my '20060102p01', 'hoge' %>
> のように # が抜けた形に変換するのですが（ソースを見て初めて知ったけど）、
> 前述の変更の際に、my-ex プラグインは # を必要とするようになったからです。
> 
> 00default.rb に定義してある標準の my プラグインでは、# はあってもなく
> てもかまいません。
> 
> my-ex.rb の変更は意図あってのことでしょうから、wiki_style.rb を変更して
>   <%=my '20060102#p01', 'hoge' %>
> のように変換するようにしようと思いますが、それでよろしいですよね？
> 
> たぶん
> http://mux03.panda64.net/diary/20061226.html#p02
> ↑これも同じ話と思われます。
> 
> # ブログだけでなく ML でも報告してくれると助かります。。。＞みなさま

なるほど，my-ex.rbの変更が原因だったのですね．
今回 zunda さんが作られた新しいプラグイン(my-sequel.rb)を試していて
問題が発生したので，原因の切り分けすらできない状況でした．

調べていただいてありがとうございます＞かずひこさん

-- 
MUNEDA Takahiro <mux03@...>

zundaです

いろいろご迷惑をおかけしてます

my-exの変更は、URLを直接書けるようにしようとしたもので、
#の無い場合に対応する必要があるとは思っていませんでした。
影響の大きさを考えると、my-ex.rb側で対応するべきだと
思います。

  date, frag = a.scan( /(\d{4}|\d{6}|\d{8}|\d{8}-\d+)(?:[^\d]*)?(?:#?([pct]\d+))?$/ )[0]

でしょうか。

手が空いたらcommitしますが、今から24時間以上かかってし
まうかもしれません。すみませんです。

--- Kazuhiko <kazuhiko@...> wrote:

> かずひこです。
> 
> my-exプラグインを有効にした上で、wikiスタイルを使うと、
> 
> 2006-12-14 zunda <zunda at freeshell.org>
>         * my-ex.rb: allow a URL as the first argument
> 
> の変更で、[[hoge|20060102#p01]] という形式でのリンクが機能しなく
> なったので、原因を調べてみました。
> 
> wiki_style.rb では、
>   [[hoge|20060102#p01]]
> を
>   <%=my '20060102p01', 'hoge' %>
> のように # が抜けた形に変換するのですが（ソースを見て初めて知ったけど）、
> 前述の変更の際に、my-ex プラグインは # を必要とするようになったからです。
> 
> 00default.rb に定義してある標準の my プラグインでは、# はあってもなく
> てもかまいません。
> 
> my-ex.rb の変更は意図あってのことでしょうから、wiki_style.rb を変更して
>   <%=my '20060102#p01', 'hoge' %>
> のように変換するようにしようと思いますが、それでよろしいですよね？
> 
> たぶん
> http://mux03.panda64.net/diary/20061226.html#p02
> ↑これも同じ話と思われます。
> 
> # ブログだけでなく ML でも報告してくれると助かります。。。＞みなさま
> 
> かずひこ
> -------------------------------------------------------------------------
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to share your
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> tDiary-devel mailing list
> tDiary-devel@...
> https://lists.sourceforge.net/lists/listinfo/tdiary-devel
> 


--------------------------------------
Yahoo! JAPAN Search Word Ranking 2006
http://pr.mail.yahoo.co.jp/search/

ただただしです。

zunda <zunda616e@...> wrote:
>my-exの変更は、URLを直接書けるようにしようとしたもので、
>#の無い場合に対応する必要があるとは思っていませんでした。
>影響の大きさを考えると、my-ex.rb側で対応するべきだと
>思います。
>
>  date, frag = a.scan( /(\d{4}|\d{6}|\d{8}|\d{8}-\d+)(?:[^\d]*)?(?:#?([pct]\d+))?$/ )[0]
>
>でしょうか。

これで問題なく動いてるっぽいので、commitしておきました。

＃ChangeLogがチョー適当なので、気が向いたら直しておいてください(笑) >zunda

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

zundaです

my-exでもspam.rbでもお騒がせしちゃってすみません。
さて作業できるようになった、と思うと直ちゃってるん
だもんなあ。みなさん、ありがとうございます。

--- TADA Tadashi <sho@...> wrote:

> ただただしです。
> 
> zunda <zunda616e@...> wrote:
> >my-exの変更は、URLを直接書けるようにしようとしたもので、
> >#の無い場合に対応する必要があるとは思っていませんでした。
> >影響の大きさを考えると、my-ex.rb側で対応するべきだと
> >思います。
> >
> >  date, frag = a.scan( /(\d{4}|\d{6}|\d{8}|\d{8}-\d+)(?:[^\d]*)?(?:#?([pct]\d+))?$/ )[0]
> >
> >でしょうか。
> 
> これで問題なく動いてるっぽいので、commitしておきました。

ありがとうございました。Wikiスタイルも使ってみないとなあ…

> ＃ChangeLogがチョー適当なので、気が向いたら直しておいてください(笑) >zunda

それでは、次に何か荒らすときにでも(マテ

ではでは、お礼まで。


--------------------------------------
Yahoo! JAPAN Search Word Ranking 2006
http://pr.mail.yahoo.co.jp/search/

ただただしです。

06/12/19 に zunda<zunda616e@...> さんは書きました:
> 理想的には設定インターフェースから拒否のログが見えるとうれしいと
> は思いますが、例えば以下の変更 (Filter::SpamFilter#debugを変更し
> ています) で、デバグログによって、どの日付の日記へのツッコミが拒
> 否されたのか簡単に知ることができるようになります。

とりあえずの実装としては良いと思います。commitしてください。

本質的には、オプション指定せずとも自動的にログを取り、最近の
拒否ログを設定画面から見られるようにするのが好ましいですね。
Todoに入れておきましょう。
＃もちろん実装してくれても良いです:-)

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

zundaです

お返事ありがとうございました。

> 06/12/19 に zunda<zunda616e@...> さんは書きました:
> > 理想的には設定インターフェースから拒否のログが見えるとうれしいと
> > は思いますが、例えば以下の変更 (Filter::SpamFilter#debugを変更し
> > ています) で、デバグログによって、どの日付の日記へのツッコミが拒
> > 否されたのか簡単に知ることができるようになります。
> 
> とりあえずの実装としては良いと思います。commitしてください。

TDIARY_VERSION = '2.1.4.20061220'としてcommitしました。

> 本質的には、オプション指定せずとも自動的にログを取り、最近の
> 拒否ログを設定画面から見られるようにするのが好ましいですね。

ですよね。

> Todoに入れておきましょう。
> ＃もちろん実装してくれても良いです:-)

そのうち〜 (できるといいなあ :P


--------------------------------------
Get the new Internet Explorer 7 optimized for Yahoo! JAPAN
http://pr.mail.yahoo.co.jp/ie/

ただただしです。

zunda <zunda616e@...> wrote:
>> とりあえずの実装としては良いと思います。commitしてください。
>
>TDIARY_VERSION = '2.1.4.20061220'としてcommitしました。

dateパラメタがないreferer付きのリクエストが来たときに死んでしまうとの報
告を受けたので、その場合は「no date」と出力するように変更しておきました。
＃テストしてません(笑)

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

NT です。

> dateパラメタがないreferer付きのリクエストが来たときに死んでしまうとの報
> 告を受けたので、その場合は「no date」と出力するように変更しておきました。
> ＃テストしてません(笑)

やっぱり、そうでしたか(苦笑)
実はあの修正でもダメでした。そこで、以下のように直してうまく動きました。

--- core/tdiary/filter/spam.rb   2006-12-27 15:40:19.213277840 +0900
+++ spam.rb 2006-12-27 23:14:24.093408808 +0900
@@ -189,7 +189,7 @@
            require 'time'
            File.open(@debug_file, 'a') do |io|
               io.flock(File::LOCK_EX)
-              io.puts "#{Time.now.iso8601}: #{@cgi.remote_addr}->#{(@cgi.params['date'][0].dump || 'no date')}: #{msg}"
+              io.puts "#{Time.now.iso8601}: #{@cgi.remote_addr}->#{((@cgi.params['date'][0].dump if @cgi.params['date'][0]) || 'no date')}: #{msg}"
            end
         end

-- 
NISHIMURA Takashi <nt@... | ni46rtk4@...>
  GnuPG: 0F7B 4211 F62A E46C 0F6C  7CBD 2861 2D77 6BDF DFDA
  Website: http://246ra.ath.cx/~takasi/

ただただしです。

"NISHIMURA Takashi" <ni46rtk4@...> wrote:
>やっぱり、そうでしたか(苦笑)
>実はあの修正でもダメでした。そこで、以下のように直してうまく動きました。
>
>--- core/tdiary/filter/spam.rb   2006-12-27 15:40:19.213277840 +0900
>+++ spam.rb 2006-12-27 23:14:24.093408808 +0900
>@@ -189,7 +189,7 @@
>            require 'time'
>            File.open(@debug_file, 'a') do |io|
>               io.flock(File::LOCK_EX)
>-              io.puts "#{Time.now.iso8601}: #{@cgi.remote_addr}->#{(@cgi.params['date'][0].dump || 'no date')}: #{msg}"
>+              io.puts "#{Time.now.iso8601}: #{@cgi.remote_addr}->#{((@cgi.params['date'][0].dump if @cgi.params['date'][0]) || 'no date')}: #{msg}"
>            end
>         end

あぁ、そうか。dumpつける位置がそのままじゃ意味がない。バカでした。
以下が正しいですね:

   (@cgi.params['date'][0] || 'no date').dump

＃こんどはテストした。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

のむらです。

追記で、
・rubyは1.8.5です。
・tdiary.confの@no_refererに以下の行を追加しましたが一緒でした。

	'^' + Regexp.quote( base_url ) + 'update\.rb.*',

以上です

Keisuke Nomura wrote:
> のむらです。
> 
> 手元のtDiaryを2.1.4.20061210(フルセット)にアップデートしたところ、
> 自分自身のupdate.rbがリンク元として記録されてしまうようになりました。
> #自分自身のアドレスでも、update.rb以外は記録されていません。

ただただしです。

06/12/12 に Keisuke Nomura<kei-nomura@...> さんは書きました:
> 手元のtDiaryを2.1.4.20061210(フルセット)にアップデートしたところ、
> 自分自身のupdate.rbがリンク元として記録されてしまうようになりました。

アップデート前のバージョンはなんでしょう?

tdiary/filterの下には何がありますか? デフォルトのフィルタ(default.rb)が
適用されてないっぽい挙動に見えます。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

のむらです。

TADA Tadashi wrote:
> アップデート前のバージョンはなんでしょう?
> 
> tdiary/filterの下には何がありますか? デフォルトのフィルタ(default.rb)が
> 適用されてないっぽい挙動に見えます。
> 

アップデート前のバージョンは、2.1.4だったとは思います。
確証はありませんが、amazon.rb関連がごにょごにょした
ときに最新版にした覚えがあります。

$ ls public_html/diary/tdiary/filter/
antirefspam.rb  default.rb  key.rb  size.rb  size.rb~  spam.rb

となりました。

ただただしです。

間があいてしまってすみません。

Keisuke Nomura <kei-nomura@...> wrote:
>アップデート前のバージョンは、2.1.4だったとは思います。
>確証はありませんが、amazon.rb関連がごにょごにょした
>ときに最新版にした覚えがあります。

だとすると、pluginの下に05referer.rbがないという可能性は?

2.1.4から現在までの間に、refererの記録をする場所が本体からプラグインに変
わっています。中途半端なアップデートをすると、更新されているはずのファイ
ルが入っていなかったりしているかも知れません。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

TADA Tadashi wrote:
> だとすると、pluginの下に05referer.rbがないという可能性は?
$ ls public_html/diary/plugin/
00default.rb  05referer.rb  10spamfilter.rb  50sp.rb  en/  ja/  zh/

ということで大丈夫そうです。

> 2.1.4から現在までの間に、refererの記録をする場所が本体からプラグインに変
> わっています。中途半端なアップデートをすると、更新されているはずのファイ
> ルが入っていなかったりしているかも知れません。

フルバージョンを丸コピーしたので大丈夫だと思います。
逆に、残っていてはまずいファイルがあるのでしょうか。

もしリンク元除外リストが適用されていないのであれば、
update.rb以外(たとえば、トップページからのリンクとか)も
記録されてしまうような気がするのですが、
それは記録されてないようなんですよね。

自己フォローです。

> もしリンク元除外リストが適用されていないのであれば、
> update.rb以外(たとえば、トップページからのリンクとか)も
> 記録されてしまうような気がするのですが、
> それは記録されてないようなんですよね。

どうもPOSTメソッドのリファラ(という表現でいいんだろうか)が
記録されているようです。
なので、update.rbとツッコミがリンク元として記録されています。
そもそもPOSTの場合はリファラを記録する必要はないと思うのですが、
なんで記録されてしまうんでしょう。

こんにちは、はしもとです。

原因は違うかも知れませんが、結果として同じ現象を経験したので、
一応ボクの場合の原因を書いてみます。

全然関係なかったらごめんなさい。
原因究明のきっかけになれば幸いです。


ボクの場合は、Rast検索を設置したときのボクのミスでした。

標準添付ではないプラグインを設置するディレクトリとして、
tdiary/custom/plugin を作っています。ここに、contribに含まれる
rast-register.rb （と言語ファイル）をコピーしました。

tdiary.conf の @options['sp.path'] には、misc/plugin、
custom/plugin の順で読み込むように指定していました。

このプラグインを設置して以来、日記を書いた後に update.rb が、
ツッコミが入った後にツッコミ対象の日記のURLが、それぞれリン
ク元に記録されるようになりました。

原因は、misc/plubin にかなり古い rast-register.rb が入っていて、
custom/plugin のものではなくこの古いファイルが読み込まれていた
からでした。

このファイルをいつ misc/plugin に入れたか覚えてませんが、過去
になんとなく contrib のプラグインをまとめて misc/plugin に入
れたことがあるような気がします。

対処として misc/plugin/rast-register.rb とその言語ファイルを
削除し、tdiary.conf の @options['sp.path'] の指定順を 
custom/plugin、misc/plugin の順に変更したら直りました。


長々と書きましたが、このように意図しない古いプラグインを読み
込んだことによって、のむらさんと同じ現象が起こっていた、と
いうことです。


-- 
HASHIMOTO Keisuke <ksk@...>

のむらです。

HASHIMOTO Keisuke wrote:
> 原因は違うかも知れませんが、結果として同じ現象を経験したので、
> 一応ボクの場合の原因を書いてみます。
> 
> 全然関係なかったらごめんなさい。
> 原因究明のきっかけになれば幸いです。

ビンゴでした！結論としてはrast検索が古いのが原因のようです。
rast-register.rbをはずしたところ、現象が出なくなりました。

最新版にすると、何故かエラーがでるのですが、
これはおいおい調べたいと思います。

ありがとうございました。

ただただしです。

06/12/19 に zunda<zunda616e@...> さんは書きました:
> 現在は、「ツッコミ中でURLを表す文字の占める割合」への入力に小数点
> が含まれていないと割合を0.0とみなすようです。下記のように、「小数
> 点と数字0文字以上」はオプションとなるのが意図した正規表現ではない
> でしょうか？

今年の7月頃、max_rateの意味をパーセントにしたので、そもそも小数で
表現する必然性はないように思えます。整数だけを受け付けるようにして
しまう方がすっきりすると思いますが、どうでしょうね?

＃はやい話が修正忘れなわけですが。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。

zundaです

「ツッコミ中でURLを表す文字の占める割合」について。個人的には
どう転んでもいいかな、と思います。

--- TADA Tadashi <sho@...> wrote:

> 今年の7月頃、max_rateの意味をパーセントにしたので、

なるほどー。まだまだウラシマ状態です。

> そもそも小数で
> 表現する必然性はないように思えます。整数だけを受け付けるようにして
> しまう方がすっきりすると思いますが、どうでしょうね?

すっきりはしますが、ほとんど実行されない plugin/10spamfilter.rb
の正規表現以外にはコストの違いはないので、今になってわざわざ限定し
てしまう必要もないかな？とも思います。

どちらかというと、max_rateが1以下ならパーセントではなくて割合と
して解釈するような機構があってもよかったな、と思いますが、
こちらはツッコミをもらうたびにコストがかかるし、
今さら感も大きいですよね。

というわけで、僕には強い意見はありません。

どうしましょう…

--------------------------------------
Get the new Internet Explorer 7 optimized for Yahoo! JAPAN
http://pr.mail.yahoo.co.jp/ie/

ただただしです。

06/12/11 に Kazuhiro NISHIYAMA<zn@...> さんは書きました:
> > >  <%%=conf_proc( "#{@cgi.params['conf'][0]}" )%>
> > > が残っているのですが [1]、こちらも直したほうがよいのですよね。
>
> 2.0.4でも残っているように見えます。

ああぁ。すみません、見落としです orz

もっとも、この部分は脆弱性にはつながらないと思います。
もし、これを使ってtDiaryを攻撃する方法を発見された方は、
このMLなど公の場ではなく、私個人宛てに教えてください。

-- 
ただただし <http://sho.spc.gr.jp/>
♪ツッコミは、短く鋭く愛を込めて。