tDiary

中川です。

	私が11月25日13時46分に発信した
	「[tDiary-devel] amazon.rb でエラー」こと
	<051125134642.M0102780@...>より

>    はじめまして、中川と申します。
>  
>  私の環境(*) において
>  1. tDiary core 2.1.3 + CVS 版の plugin
>  2. tDiary core 2.1.3 + 2.1.3 plugin ( tdiary-full-2.1.3 から)
>  3. tDiary core 2.0.2 + CVS 版の plugin
>  
>  で、それぞれ amazon.rb を使っているとエラーが出ます。
>  
>  (*) -> Apache 1.3.32 + SuExec で CGI 
>         Ruby は 1.8.3 でエラーが出たところで 1.8.2 にダウングレード
>         したけれども同様

...書き忘れていましたね。上記環境は FreeBSD 4-stable (4.11-STABLE #10)
です

  私のデスクトップ環境 ( FreeBSD 5-stable こと 5.4-STABLE #12 )
の上に, 上記環境と同じ version の構成で SuExec を外したものを
インストールして、tdiary.conf や データを持ってきて実験しました。

  結果として、SuExec なしならば amazon.rb は動く、ということが
検証できたと思っています。


--
  中川 恒雄 ( T.Nakagawa )         mailto:yaemon@...
                                   http://www.kikansha.jp/~yaemon/

中川です。

すみません、tDiary-devel ではなく tDiary-users レベルの
話でした。

	私が11月25日18時52分に発信した
	「Re: [tDiary-devel] amazon.rb でエラー」こと
	<051125185229.M0166239@...>より

>    私のデスクトップ環境 ( FreeBSD 5-stable こと 5.4-STABLE #12 )
>  の上に, 上記環境と同じ version の構成で SuExec を外したものを
>  インストールして、tdiary.conf や データを持ってきて実験しました。
>  
>    結果として、SuExec なしならば amazon.rb は動く、ということが
>  検証できたと思っています。

<>

  desktop機 で SuExec 環境を作るのに手間取って、今まで原因究明が
遅れました。

  結果は、ふつうの環境ならば SuExec でも amazon.rb に問題なしです。


  ...それで httpd.conf に問題があると判断して、
今まで使っていた httpd.conf と httpd.conf.default の差分を取って
最新の httpd.conf-dist にマージしていたのですが

  結果は単純なことでした。

	「[tDiary-devel] amazon.rb でエラー」こと
	<051125134642.M0102780@...>より

>  loading from unsafe path .:/usr/local/lib/ruby/site_ruby/1.8:
>      /usr/local/lib/ruby/site_ruby/1.8/i386-freebsd4:
>      /usr/local/lib/ruby/site_ruby:/usr/local/lib/ruby/1.8:
>      /usr/local/lib/ruby/1.8/i386-freebsd4:. (SecurityError)

<>

>  loading from unsafe path .:/usr/local/lib/ruby/site_ruby/1.8:
>      /usr/local/lib/ruby/site_ruby/1.8/i386-freebsd4:
>      /usr/local/lib/ruby/site_ruby:/usr/local/lib/ruby/1.8:
>      /usr/local/lib/ruby/1.8/i386-freebsd4:. (SecurityError)

"." が SuExec 的に unsafe でした。

私自身、すっかり忘れていたのですが、public_html が
$HOME/MyWeb/severname へのシンボリックリンクだったのです。

  それで $HOME/public_html 以下にない path から読み込もうとしている
と判断されて unsafe と言われていたのでした。


  どうも失礼しました。


--
  中川 恒雄 ( T.Nakagawa )         mailto:yaemon@...
                                   http://www.kikansha.jp/~yaemon/

YAAです。

At Mon, 21 Nov 2005 14:43:34 +0900,
Kazuhiko wrote:

> tDiary-Rast を、関西オープンソース 2005 の Rast の発表でデモンストレーショ
> ンしたもの（第二で動いているものともいう）にアップデートしました。

ありがとうございます。
導入してみました。
CVS 最新の、以下のような環境に導入しました。
  Powered by Rast 0.3.1
  Generated by tDiary version 2.1.3.20051021
  Powered by Ruby version 1.8.2 

manual どおりに rast-register.rb を plugin directory に copy してから
index の作成を行なうと、error になりました。plugin directory に
rast-register.rb が無い状態だと index が作成できました。

また、rast-search.rb も同様に、rast-register.rb が plugin directory に
あると error になりました。rast-register.rb が plugin directory にない
状態だと正常に検索できます。

そして、rast-register.rb が plugin directory にある状態で日記を書いて
も index は作成されませんでした。

以下、そのときの error です。
---- rast-search.rb の error ----
Plugin Error

Errors in plugins? Retry to Update or Configure.

    TDiary::PluginError
    Plugin error in 'rast-register.rb'.
    uninitialized constant TDiary::Plugin::Rast
    (plugin/rast-register.rb):72:in `instance_eval'


---- rast-register.rb の error ----
% ruby plugin/rast-register.rb
(2002/) Plugin error in 'rast-register.rb'.
uninitialized constant TDiary::Plugin::Rast
(plugin/rast-register.rb):72:in `instance_eval'
./tdiary.rb:658:in `initialize'
plugin/rast-register.rb:144:in `new'
plugin/rast-register.rb:144:in `initialize'
plugin/rast-register.rb:244:in `new'
plugin/rast-register.rb:244:in `initialize'
plugin/rast-register.rb:243:in `each'
plugin/rast-register.rb:243:in `initialize'
plugin/rast-register.rb:242:in `transaction'
plugin/rast-register.rb:242:in `initialize'
plugin/rast-register.rb:241:in `each'
plugin/rast-register.rb:241:in `initialize'
plugin/rast-register.rb:239:in `each'
plugin/rast-register.rb:239:in `initialize'
plugin/rast-register.rb:238:in `transaction'
plugin/rast-register.rb:119:in `open'
plugin/rast-register.rb:119:in `transaction'
plugin/rast-register.rb:238:in `initialize'
plugin/rast-register.rb:289:in `new'
plugin/rast-register.rb:289
zsh: exit 1     ruby plugin/rast-register.rb


以上、報告のみですが、よろしくお願いします。

-- 
YAA  =A=
$$ "You say this is Earl Grey? I'd swear that it was Darjeeling." $$

かずひこです。

At Wed, 23 Nov 2005 20:00:08 +0900,
YAA wrote:

> manual どおりに rast-register.rb を plugin directory に copy してから
> index の作成を行なうと、error になりました。plugin directory に
> rast-register.rb が無い状態だと index が作成できました。
> 
> また、rast-search.rb も同様に、rast-register.rb が plugin directory に
> あると error になりました。rast-register.rb が plugin directory にない
> 状態だと正常に検索できます。

ご報告ありがとうございます。autoload ではなく require を使うようにしてコ
ミットしました。これで動くでしょうか？

diff -u -r1.1 -r1.2
--- rast-register.rb    21 Nov 2005 05:33:09 -0000      1.1
+++ rast-register.rb    23 Nov 2005 11:05:44 -0000      1.2
@@ -3,7 +3,7 @@
 # Copyright (C) 2005 Kazuhiko <kazuhiko@...>
 # You can redistribute it and/or modify it under GPL2.
 #
-autoload(:Rast, 'rast')
+require 'rast'
 
 mode = ""
 if $0 == __FILE__
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

かずひこです。

従来 rast-register.rb を CGI として実行することでインデックスの一括登録が
できるようになっていましたが、tDiary の設定画面で行えるように変更し、従来
の CGI モードを廃止しました (コマンドラインによる一括登録はできます)。

なお、例えばレンタル日記環境などで、ユーザによるインデックスの一括登録を
(負荷対策のため) 禁止したいケースでは

  @options['rast_register.hideconf'] = true

を指定してください。

いまのところメッセージが英語だけですが、そのへんについては今後修正してい
きたいと思います。
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

YAAです。

At Wed, 23 Nov 2005 20:07:22 +0900,
Kazuhiko wrote:

> > manual どおりに rast-register.rb を plugin directory に copy してから
> > index の作成を行なうと、error になりました。plugin directory に
> > rast-register.rb が無い状態だと index が作成できました。
> > 
> > また、rast-search.rb も同様に、rast-register.rb が plugin directory に
> > あると error になりました。rast-register.rb が plugin directory にない
> > 状態だと正常に検索できます。
> 
> ご報告ありがとうございます。autoload ではなく require を使うようにしてコ
> ミットしました。これで動くでしょうか？

ありがとうございます。
index の作成はできるようになりました。
しかし全ての index を作り終えた後で error が出ました。

% ruby plugin/rast-register.rb
(2002/) 0611 0612 0613 0614 0615 0616 0617 0618 0619 0620 0621
(snip)
9 1120 1121 1122 1123 1124
plugin/rast-register.rb:303: undefined method `[]' for nil:NilClass (NoMethodError)
zsh: exit 1     ruby plugin/rast-register.rb

@conf が nil なため error になっているようです。


At Thu, 24 Nov 2005 01:09:18 +0900,
Kazuhiko wrote:

> 従来 rast-register.rb を CGI として実行することでインデックスの一括登録が
> できるようになっていましたが、tDiary の設定画面で行えるように変更し、従来
> の CGI モードを廃止しました (コマンドラインによる一括登録はできます)。

こちらを使った場合も index は作成されました。



上記の問題とは別に、tlink.rb を使っている場合に
command line からの index 作成、および
日記更新時の index 作成に失敗します。

以下がその際に出る error です。

Plugin Error

Errors in plugins? Retry to Edit(2002-06-11) or Configure.

    TDiary::PluginError
    Plugin error in '50sp.rb'.
    Plugin error in 'plugin2/tlink.rb'.
    undefined method `+' for nil:NilClass
    (plugin/50sp.rb):162:in `instance_eval' 

よろしくお願いします。

-- 
YAA  =A=
$$ "You say this is Earl Grey? I'd swear that it was Darjeeling." $$

かずひこです。

ご報告ありがとうございます。

At Thu, 24 Nov 2005 21:04:17 +0900,
YAA wrote:

> index の作成はできるようになりました。
> しかし全ての index を作り終えた後で error が出ました。
> 
> % ruby plugin/rast-register.rb
> (2002/) 0611 0612 0613 0614 0615 0616 0617 0618 0619 0620 0621
> (snip)
> 9 1120 1121 1122 1123 1124
> plugin/rast-register.rb:303: undefined method `[]' for nil:NilClass (NoMethodError)
> zsh: exit 1     ruby plugin/rast-register.rb
> 
> @conf が nil なため error になっているようです。

プラグイン関連の処理をコマンドライン時に実行しないように変更しました。

> 上記の問題とは別に、tlink.rb を使っている場合に
> command line からの index 作成、および
> 日記更新時の index 作成に失敗します。
(snip)
>     Plugin error in '50sp.rb'.
>     Plugin error in 'plugin2/tlink.rb'.
>     undefined method `+' for nil:NilClass
>     (plugin/50sp.rb):162:in `instance_eval' 

@plugin を作成する際に cache_path をセットしていないのが原因でした。こち
らも修正してコミットしました。
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

YAAです。

At Thu, 24 Nov 2005 21:26:48 +0900,
Kazuhiko wrote:

> プラグイン関連の処理をコマンドライン時に実行しないように変更しました。
> 
> > 上記の問題とは別に、tlink.rb を使っている場合に
> > command line からの index 作成、および
> > 日記更新時の index 作成に失敗します。
> (snip)
> >     Plugin error in '50sp.rb'.
> >     Plugin error in 'plugin2/tlink.rb'.
> >     undefined method `+' for nil:NilClass
> >     (plugin/50sp.rb):162:in `instance_eval' 
> 
> @plugin を作成する際に cache_path をセットしていないのが原因でした。こち
> らも修正してコミットしました。

ありがとうございました。
正常に動作するようになりました。

-- 
YAA  =A=
$$ "You say this is Earl Grey? I'd swear that it was Darjeeling." $$

かずひこです。

At Thu, 17 Nov 2005 23:34:49 +0900,
TADA Tadashi wrote:

> image.rbプラグインで、画像のアップロード時にimage_infoメソッド内でエラー
> になり、登録できない画像が報告されました。このメールに添付してあります。
> 
> JPEGフォーマットに詳しくないので原因がよくわからないんですが、どなたか詳
> しい方、回避策などわかりませんか?

同じくフォーマットには詳しくないけど、データ読み込みの条件判定がおかしい
のを修正したら動くようになりましたのでコミットしました。
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

ただただしです。

On Fri, 18 Nov 2005 00:39:33 +0900
Kazuhiko <kazuhiko@...> wrote:
>同じくフォーマットには詳しくないけど、データ読み込みの条件判定がおかしい
>のを修正したら動くようになりましたのでコミットしました。

こんな単純な話だったとは……とほほ。
Stable-2_0にもあてておきました。

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

KASHIJUKUです。

image.rbプラグインのimage_infoメソッドには、もう一つ問題があって、
gif, png, jpegのすべてで、heightとwidthの扱いが逆になっています。

で、なぜこれで正しく動作するのか不思議だったのですが、メソッドが

    return image_type, image_height, image_width
                             ~~~~~~        ~~~~~
と返しているのを、呼び出し元が、

    img_type, img_w, img_h = open(File.join(@image_dir,img).untaint,
              ~~~~~  ~~~~~         'r') {|f| image_info(f)}

と受けているので、最後のつじつまは合っているのでした。

ただ、このままでは少し気持ち悪いので、添付のパッチ(image_info の height
とwidthをひっくり返したもの）を当てておくことを提案します。

かずひこです。

At Wed, 09 Nov 2005 23:43:30 +0900,
MoonWolf wrote:

> Subject: Re: [tDiary-devel] plugin/rast-register.rb: 例外処理
(snip)
> 'rast'と'htree'のrequireでエラーになるので例外処理追加しました。

そのへんが require できなかったらそもそも使えないプラグインなので、単にプ
ラグインを有効にしなけりゃいいだけだと思うのですが、こういう処理って必要
なのでしょうか？
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

ただただしです。

On Thu, 10 Nov 2005 09:27:31 +0900
Kazuhiko <kazuhiko@...> wrote:
>そのへんが require できなかったらそもそも使えないプラグインなので、単にプ
>ラグインを有効にしなけりゃいいだけだと思うのですが、こういう処理って必要
>なのでしょうか？

contribならそれでもいいけど、pluginに入っている以上、そういう配慮は必要
かも知れません。coreとpluginはいちおう、動作環境として標準状態のrubyだけ
を求めているので。

＃そういう意味では、rast-register.rbがpluginに入っているのはルール違反かも

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

かずひこです。

At Thu, 17 Nov 2005 23:13:54 +0900,
TADA Tadashi wrote:

> >そのへんが require できなかったらそもそも使えないプラグインなので、単にプ
> >ラグインを有効にしなけりゃいいだけだと思うのですが、こういう処理って必要
> >なのでしょうか？
> 
> contribならそれでもいいけど、pluginに入っている以上、そういう配慮は必要
> かも知れません。coreとpluginはいちおう、動作環境として標準状態のrubyだけ
> を求めているので。
> 
> ＃そういう意味では、rast-register.rbがpluginに入っているのはルール違反かも

contrib/plugin/README.ja:

| ここに収録されたプラグインは、
| 
| * 汎用性が低い
| * 国際化対応されていない(する意味がない)
| * tDiaryのバージョンに合わせたアップデートがされていない
| 
| 等の理由でプラグイン集への収録が見送られているものですが、
| それぞれはとても有用なプラグインです。

ということなので、「汎用性が低い」扱いで contrib 行きがよろしいでしょうか？
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

ただただしです。

Kazuhiko <kazuhiko@...> wrote:
>> ＃そういう意味では、rast-register.rbがpluginに入っているのはルール違反かも
>
>contrib/plugin/README.ja:
>
>| ここに収録されたプラグインは、
>| 
>| * 汎用性が低い
>| * 国際化対応されていない(する意味がない)
>| * tDiaryのバージョンに合わせたアップデートがされていない
>| 
>| 等の理由でプラグイン集への収録が見送られているものですが、
>| それぞれはとても有用なプラグインです。
>
>ということなので、「汎用性が低い」扱いで contrib 行きがよろしいでしょうか？

そうですね。そもそも他のツールはcontribに入っているのに、rast-register.rb
だけpluginに入っているのも変なので、contribに移しましょう。

＃util/rast-searchにコピればいいよね? こっちでやっておきます。

README.jaの方は「coreおよびpluginの動作条件にマッチしないもの」という条
件を加えるべきですね。これもこっちでやっておきます。

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

かずひこです。

At Mon, 21 Nov 2005 13:54:37 +0900,
TADA Tadashi wrote:

> >ということなので、「汎用性が低い」扱いで contrib 行きがよろしいでしょうか？
> 
> そうですね。そもそも他のツールはcontribに入っているのに、rast-register.rb
> だけpluginに入っているのも変なので、contribに移しましょう。
> 
> ＃util/rast-searchにコピればいいよね? こっちでやっておきます。

contrib/plugin/ かと思っていましたが、どうせセットで使うんだし、おっしゃ
るように contrib/util/rast-search/ でいいですね。

ついでにいろいろ変更してコミットしたいので、私が作業します。
-- 
かずひこ <http://wiki.fdiary.net/kazuhiko/>
  「恋とハックはアジャイルが命！」

ただただしです。

ちょっと多忙で放置してしまいました。すみません。

On Wed, 09 Nov 2005 23:39:43 +0900
MoonWolf <moonwolf@...> wrote:
>plugin/image.rbでimagesディレクトリが無い場合にエラーが発生するので、
>resuceするようにしました。

そのままcommitしました。

＃本当はimagesがないことをメッセージで伝えたほうがいいんだけど、
＃image_gpsとかにまで影響が出てしまったのでペンディング。

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