tDiary

きたです．
こんにちは．

>うちの環境では以下のように tdiary.rb のあるパスを untaint しないと動
>かないのですが、なにか間違ってますか?
>
>環境は
>* ruby CVS HEAD
>* tDiary CVS HEAD
>* mod_ruby CVS HEAD
>で、index.rb は tdiary.rb とは別のディレクトリから Symlink を張ってある
>状態 (i.e. org_path != '.') です。

ruby の HEAD をおいかけてないので分からないのですが，もしかして汚染され
るオブジェクトに変更があったということでしょうか?

今後リリースされる ruby でも同じ状況なら，頂いたパッチを適用した方がよさ
そうですね．

mput です。

On 2003.04.30, at 22:51, Junichiro Kita wrote:

> ruby の HEAD をおいかけてないので分からないのですが，もしかして汚染され
> るオブジェクトに変更があったということでしょうか?

File::readlink は以下のようになってるんですが、

static VALUE
rb_file_s_readlink(klass, path)
     VALUE klass, path;
{
#ifdef HAVE_READLINK
     char *buf;
     int size = 100;
     int rv;
     VALUE v;

     SafeStringValue(path);
     buf = xmalloc(size);
     if ((rv = readlink(RSTRING(path)->ptr, buf, size)) == size) {
	size *= 2;
	buf = xrealloc(buf, size);
     }
     if (rv < 0) {
	free(buf);
	rb_sys_fail(RSTRING(path)->ptr);
     }
     v = rb_tainted_str_new(buf, rv);
     free(buf);

     return v;
#else
     rb_notimplement();
     return Qnil;		/* not reached */
#endif
}

rb_tainted_str_new で最初から汚染されているようです。
# symlink の先は問答無用で信用ならないって事でしょうかね。

この変更が一時的なものなのかはちょっと分かりません (cvs log からは検索
できませんでした)。

  --
mput @ OTSUKARESAMA.
mailto:root@mput.dip.jp
http://mput.dip.jp

yoshimiです。

新しいものを、
http://shimoi.s26.xrea.com/files/image_plugin.rb
に置きました。
とりこんでください。

From: TADA Tadashi <sho@spc.gr.jp>
Subject: [tDiary-devel] 絵日記
Date: Tue, 22 Apr 2003 23:32:42 +0900
Message-ID: <20030422233242.4022fb05.sho@spc.gr.jp>

>    * imgタグにはclass="photo"を入れたい(posttdiary.rbとの共通化) 

これはやっておきましたが、
image_left(num,'hoge')、image_right(num,'hoge')というのを
新設したので、image_left(num,'hoge')を使った場合
class="photo left"
のようになります。問題ありますでしょうか？


>    * imgタグの閉じタグは「 />」→「>」へ(XHTMLじゃないので) 

直しました。


>    * formのレイアウト微調整

どういうのがいいのかわからないのでやっちゃってください。


>    * @optionsにデフォルト値を設定(両方「./image/」でいいと思う)

やっておきました。


>    * セキュアモードでも動くように

これは僕には無理です(^^;


>    * 1日あたりの画像枚数を制限できるように
>    * 1枚あたりのサイズ制限ができるように

こちら近いうちにやっておきますが、よかったらやっちゃってくだ
さい。


1日3枚だと10年後に10000枚越えるので、機能追加版のほうではファ
イルを置くディレクトリ年毎に作るオプションというのも付けてい
ます。

ただです。

Yoshimi KURUMA <yoshimik@iris.dti.ne.jp> wrote:
>新しいものを、
>http://shimoi.s26.xrea.com/files/image_plugin.rb
>に置きました。
>とりこんでください。

どうもです。daiさんにライセンスの確認だけ取ってから、CVSに入れます。

>image_left(num,'hoge')、image_right(num,'hoge')というのを
>新設したので、image_left(num,'hoge')を使った場合
>class="photo left"
>のようになります。問題ありますでしょうか？

img.right、img.leftはamazonが生成していますね。ですから、それを活用する
のであれば、photoは付けない方がいいと思います。両方付けると混乱する気が
しますし。

>>    * formのレイアウト微調整
>
>どういうのがいいのかわからないのでやっちゃってください。

CVSに取り込んでからぼちぼちやります。

>>    * セキュアモードでも動くように
>
>これは僕には無理です(^^;

これも近々トライしてみます。

>1日3枚だと10年後に10000枚越えるので、機能追加版のほうではファ
>イルを置くディレクトリ年毎に作るオプションというのも付けてい
>ます。

標準でこれでもいいように思いますけどねー。互換性を考えると難しい?

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

ただただしです。

image.rbとして、commitしておきました(ファイル名の「_plugin」は自明
なので外しました)。まだあまりいじっていませんが、classのところだけは
変えてあります。

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

ただです。

image.rbをセキュアモードでも動くようにしてみました。ただ、複数のファイル
フォーマットをサポートしようとするとかなり性能面で無理がありそうなので、

		セキュアモードでは*.jpgのみ!!

という大胆な割り切りをするようにしてあります:-)

あと、yoshimiさん(?)が仕込みかけてあったmaxnumとmaxsizeも効くようにした
つもりです(Tempfileクラスは1.6の途中で仕様が変わってるんですねぇ)。これ
らのデフォルト値は議論の余地があるかも知れません。

それからフォームのチューニングもしました。追加のフォームを後ろに回したり、
一覧をtableにしたり、けっこういじってしまった。すみません。作業の流れ的
にこんな感じがいいんじゃないかと思います。どうでしょう?

＃他にもかなりいじってしまったので、エンバグしてるかも

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

ただです。

TADA Tadashi <sho@spc.gr.jp> wrote:
>＃他にもかなりいじってしまったので、エンバグしてるかも

やっぱりバグってたので直しました。とほほ。

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

よしみです。

From: TADA Tadashi <sho@spc.gr.jp>
Subject: [tDiary-devel] Re: 絵日記
Date: Thu, 24 Apr 2003 17:57:10 +0900
Message-ID: <20030424175112.EE04.SHO@spc.gr.jp>

> あと、yoshimiさん(?)が仕込みかけてあったmaxnumとmaxsizeも効くようにした
> つもりです(Tempfileクラスは1.6の途中で仕様が変わってるんですねぇ)。これ
> らのデフォルト値は議論の余地があるかも知れません。

制限値は、普通に使っていればまず引掛らないであろう値にしただ
けなんですが、もっと小さい値にしておいてセーフモードのときの
み制限するというのもありかもしれません。
(Ruby1.6.7 -> 1.6.8で仕様変更があると思ってなかったんですが、
daiさんにだめだしされました。)


> それからフォームのチューニングもしました。追加のフォームを後ろに回したり、
> 一覧をtableにしたり、けっこういじってしまった。すみません。作業の流れ的
> にこんな感じがいいんじゃないかと思います。どうでしょう?

テーブルにするのであれば、画像の上に
<%=image 0,'title0'%>
みたいなものを表示してあると初心者にも優しいと思います。
機能追加版に実装してあります。
最初はいらない機能だと思っていたんですが、書き間違いが減るの
と相俟って(数少ない)利用者からは好評です。
スタイル依存になりますが、新しいスタイルが出ても2、3行の追加
で対応できますし。
http://www.iris.dti.ne.jp/~yoshimik/tdiary-file/newadd.jpg


> ＃他にもかなりいじってしまったので、エンバグしてるかも

とりあえず一箇所みつけました。

Index: image.rb
===================================================================
RCS file: /cvsroot/tdiary/plugin/image.rb,v
retrieving revision 1.3
diff -u -r1.3 image.rb
--- image.rb	24 Apr 2003 09:42:07 -0000	1.3
+++ image.rb	24 Apr 2003 10:26:58 -0000
@@ -128,7 +128,7 @@
 			rescue NameError
          	size = @cgi.params['plugin_image_file'][0].stat.size
 			end
-         if images.length < maxnum and size <= maxsize then
+         if images.compact.length < maxnum and size <= maxsize then
             file = "#{@image_dir}/#{date}_#{images.length}.#{extension}".untaint
 	         File::umask( 022 )
 	         File::open( file, "wb" ) do |f|

ただただしです。

On Thu, 24 Apr 2003 19:59:49 +0900 (JST) Yoshimi KURUMA <yoshimik@iris.dti.ne.jp> wrote:
>制限値は、普通に使っていればまず引掛らないであろう値にしただ
>けなんですが、もっと小さい値にしておいてセーフモードのときの
>み制限するというのもありかもしれません。

そうですね。それは考えました。セキュアモードではmaxnumを1、maxsizeを
10000……じゃ厳しすぎるかしらん? 

>(Ruby1.6.7 -> 1.6.8で仕様変更があると思ってなかったんですが、
>daiさんにだめだしされました。)

どこであったか調べてないんですが、いつの時点からかTempfile#sizeが加
わっています。うちの1.6.5にはないので、Tempfile.statを使うようにして
あります。

>テーブルにするのであれば、画像の上に
><%=image 0,'title0'%>
>みたいなものを表示してあると初心者にも優しいと思います。
>機能追加版に実装してあります。

これは見知っていたんですが、どうせならボタンを押すと上のtextareaに該
当する文字列が追加されるようなJavaScriptだとカッコイイかも。誰か書き
ませんかね:-) 

>> ＃他にもかなりいじってしまったので、エンバグしてるかも
>
>とりあえず一箇所みつけました。

これは帰りの電車で気がついたので(^^;、手元ではもう直しました。image_list
を整理しているうちにうっかり削ってしまった。あとでコミっておきます。

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

よしみです。

Message-ID: <20030424235644.199e8a2d.sho@spc.gr.jp>

> これは見知っていたんですが、どうせならボタンを押すと上のtextareaに該
> 当する文字列が追加されるようなJavaScriptだとカッコイイかも。誰か書き
> ませんかね:-) 

以前この要望をもらって調べた結果、textareaのカーソル位置に
挿入するのはIE独自のtextRangeというものを使わないと無理なよ
うだったので、たんに上に表示するようにしていました。

とりあえずIE以外でも動く、本文の最後に挿入するものを作ってみ
ました。
http://shimoi.s26.xrea.com/files/image2.rb

ただです。

Yoshimi KURUMA <yoshimik@iris.dti.ne.jp> wrote:
>とりあえずIE以外でも動く、本文の最後に挿入するものを作ってみ
>ました。
>http://shimoi.s26.xrea.com/files/image2.rb

おー、good job!!

これで十分だと思いますよ。一番よく使うシーンは、本文末尾近くへの追加でしょ
うし。この線で、少し調整してからcommitしておきます。maxnum、maxsizeの制
限も。

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

yoshimiです。

> 日記の方にも書きましたが<http://sho.tdiary.net/20030415#p03>、更新フォー
> ムに埋め込まれるedit_procプラグインを実装しました。
> 
> これによって、フォームの要素を埋め込むことで、更新のタイミングでいろいろ
> 追加できるようになります。他の既存の要素とは、name属性に「plugin_プラグ
> イン名_xxx」のように指定して名前空間で分離します。

update.rhtml等でのedit_procの埋め込みですが、edit_proc2とか
適当なものを作って、登録・プレビューボタンの後に埋め込んでい
ただけませんか？

これがあると、tdiary-usersで要望のあった、
「更新画面にスタイルに応じた書式を表示」
のようなものが奇麗にできると思うんですが。

ただです。

Yoshimi KURUMA <yoshimik@iris.dti.ne.jp> wrote:
>これがあると、tdiary-usersで要望のあった、
>「更新画面にスタイルに応じた書式を表示」
>のようなものが奇麗にできると思うんですが。

その要望は私も読んだんですが、本当に必要なのかと考えると、ちょっと疑問だ
なぁと感じています。

「日記の書式」に関しては、たぶんWikiの編集画面からの類推で要望を出したん
じゃないかと思うのですが、日記とWikiでは想定ユーザが違いすぎるだろうと。
日記は毎日書くものだから何日かすれば覚えられるはずで、誰がいつ訪れて書い
てくれるかわからないWikiとは意味が違いますよね。

だったら、今のスタイル名をリンクにして、どこかのサイトに用意してある書式
解説のページに飛べるようにした方が良いと思います。ふむ、これはアリかも。

「プラグインの使い方」はたしかにあるといいかも知れないけど、それを実現し
ようと思ったら、こんどは各プラグインから書式情報を提供してもらわないとい
けないので、かなりの手間が予想されます。

＃国際化やスタイル対応まで想定すると気が遠くなります……(笑)

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

きたです．

> だったら、今のスタイル名をリンクにして、どこかのサイトに用意してある書式
> 解説のページに飛べるようにした方が良いと思います。ふむ、これはアリかも。

実は，misc/style/スタイル名/README というファイルがあったら pre で囲っ
て include してしまう，という update.rhtml をコッソリ作ってたんだけど，
リンクにしてどっかに飛ばす方がいいっすね．


> 「プラグインの使い方」はたしかにあるといいかも知れないけど、それを実現し
> ようと思ったら、こんどは各プラグインから書式情報を提供してもらわないとい
> けないので、かなりの手間が予想されます。
> 
> ＃国際化やスタイル対応まで想定すると気が遠くなります……(笑)

同感．

こんにちは、森脇です。

On Sun, 6 Apr 2003 00:03:09 +0900
TADA Tadashi <sho@spc.gr.jp> wrote:

> theme_urlはプラグインです。ですから上書き定義ができます。
> 例えばtheme_url.rbというプラグインファイルを用意して、
> 
>    def theme_url
>       'http://example.com/~tdiary/theme'
>    end
> 
> のように書けばヨイのです。tDiary.Netで使っているのと同じ
> 手法です。

これで運用できるのですが、設定ページでテーマを指定する際、プルダウンから選択することができず多少不便です。
この不便さは解消できますでしょうか。技術的に、プラグインで可能なことでしょうか。

では。

-----
森脇 大悟  beatles@sgtpepper.net

Nanaです。

On Sun, 6 Apr 2003 18:54:28 +0900
Daigo Moriwaki <techml@sgtpepper.net> wrote:

> こんにちは、森脇です。
 これで運用できるのですが、設定ページでテーマを指定する際、プルダウンから選択することができず多少不便です。
> この不便さは解消できますでしょうか。技術的に、プラグインで可能なことでしょうか。

私はsakuraweb(レンタルサーバー)でtDiaryを複数設置していますが、
theme_url.rbを使い設定画面のプルダウンでテーマの選択はできています。
一応参考になればと思い投稿してみました。

ただただしです。

On Sun, 6 Apr 2003 18:54:28 +0900 Daigo Moriwaki <techml@sgtpepper.net> wrote:
>>    def theme_url
>>       'http://example.com/~tdiary/theme'
>>    end
>> 
>> のように書けばヨイのです。tDiary.Netで使っているのと同じ
>> 手法です。
>
>これで運用できるのですが、設定ページでテーマを指定する際、プルダウンから選択することができず多少不便です。
>この不便さは解消できますでしょうか。技術的に、プラグインで可能なことでしょうか。

設定ページに出てくる一覧は、tDiaryのインストールディレクトリ、つまり
tdiary.rbと同じディレクトリにあるthemeディレクトリの下にあるテーマの
一覧です。

ですから、まずテーマはここに入れる。そしてそこを指すようにtheme_url
プラグインを書く、というのが現時点での正しいスタイルになります。

ただ、この方法だとtDiaryのインストールディレクトリをWebで公開できる
場所にしなくてはならないので、それを回避するための考慮は必要かも知れ
ません。現時点では発想を逆にして、Webで公開できる場所にテーマを集め、
そのディレクトリをtDiaryインストールディレクトリにシンボリックリンク
をはる、という手法がありかも。

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

こんにちは、森脇です。

On Sun, 6 Apr 2003 22:17:28 +0900
TADA Tadashi <sho@spc.gr.jp> wrote:

> 設定ページに出てくる一覧は、tDiaryのインストールディレクトリ、つまり
> tdiary.rbと同じディレクトリにあるthemeディレクトリの下にあるテーマの
> 一覧です。

なるほど。
Web公開用に対するsymlinkをインストールディレクトリに作り、うまく機能しました。
便利になりました。ありがとうございます。

さっそく、Debianパッケージも修正し、公開しました。

では。

-----
森脇 大悟 beatles@sgtpepper.net