ruby-dev

まつもと ゆきひろです

In message "[ruby-dev:23890] Re: cgi, cgi/session"
    on 04/07/14, Yukihiro Matsumoto <matz@ruby-lang.org> writes:

||次の点を修正するパッチを書きました。よろしかったら取り入れてください。
||
||　・Session#delete()がエラーになることがある
||　・妥当性を確認せずにuntaintするのはどうかと
||  ・空のフォームデータでエラーになることがある
||　・CGI#[]をもっとシンプルに
||
||read_multipart()は、きちんと書き直したほうがよさそうですが、単にエラー
||回避のコードを足しました。
|
|最近、青山さんから反応がないので、どうしましょうね。私が管理
|しているcgi/session については取り込むことに異存はないですが。

と書いてから、改めて添付のパッチを読んでみました。

　・Session#delete()がエラーになることがある
　・妥当性を確認せずにuntaintするのはどうかと

これは妥当だと思います。取り込みます。

  ・空のフォームデータでエラーになることがある

このパッチには2というマジックナンバーを含むんですが、これは
もしかしてEOL.sizeの意味ですか。内容が妥当かどうかは私には判
断できませんでした。

　・CGI#[]をもっとシンプルに

さりげなく書いてあるんですが、これってかなり規模の大きい変更
ですよね(CGI#[]が単なる文字列を返す)。これっていうのは、おそ
らくは[ruby-list:39021]の意見の延長線上だと思いますが、影響
範囲が大きすぎてちょっと受け入れがたいです。

HEADの方では CGI#[] は CGI::Valueモジュールでextendした文字
列を返すようにしています。バックポートするのを忘れてましたが、
こちらの方で手は打ちませんか。少なくとも[ruby-list:39021]の
要求は満足すると思うのですが。

                                まつもと ゆきひろ /:|)

Thread

Prev Next

In This Thread

Prev Next

[#23843] st_dev by stat() != st_dev by fstat() in windows — "H.Yamamoto" <ocean@...2.ccsnet.ne.jp>

[#23854] class in singleton class — "NAKAMURA, Hiroshi" <nakahiro@...>

[#23855] Are these comittable? — "H.Yamamoto" <ocean@...2.ccsnet.ne.jp>

[#23859] arguments to Thread.start — nobu.nakada@...

[#23864] avalue and break in C block — nobu.nakada@...

[#23871] syntax error — WATANABE Hirofumi <eban@...>

[#23876] cgi, cgi/session — 堀川 久 <vzw00011@...>

[#23883] [bug] strtod(".") should raise ArgumentError — "H.Yamamoto" <ocean@...2.ccsnet.ne.jp>

[#23884] Ruby 1.8.2 preview1にむけて — matz@... (Yukihiro Matsumoto)

[#23888] super in Method — nobu.nakada@...

[#23891] openssl test failed — MIYAMUKO Katsuyuki <k-miyamuko@...>

[#23922] ruby 1.8.2 preview1 — matz@... (Yukihiro Matsumoto)

[#23946] [mswin32][ruby1.8.2] $?.pid == 0 — Masahiro Kitajima <katonbo@...>

[#23963] bcc32 {f,fd,fs}open — "H.Yamamoto" <ocean@...2.ccsnet.ne.jp>

[#23965] WEBrick::HTTPUtils.escape(" ") — Tanaka Akira <akr@...17n.org>

[#23967] WEBrick::HTTPUtils.escape and URI.escape — Tanaka Akira <akr@...17n.org>

[#23970] ruby_1_8 のバイナリ互換 — "H.Yamamoto" <ocean@...2.ccsnet.ne.jp>

[#23971] typo in inf-ruby.el — Kazuo Saito <ksaito@...>

[#23973] 引数の上書きとsuper — matz@... (Yukihiro Matsumoto)

[#23977] テスト4 — Shugo Maeda <shugo@...>

[#23979] Re: [ruby-cvs] ruby: * eval.c (rb_eval): copy on write for argument local variable — "U.Nakamura" <usa@...>

[#23984] tempfile not removed. — Tanaka Akira <akr@...17n.org>

[#23989] core dump with calling private concat inherited from Array — Tanaka Akira <akr@...17n.org>

[#23991] 1.8.2-preview2? — akira yamada / やまだあきら <akira@...>

[#23995] String#each -> String#each_char — Shugo Maeda <shugo@...>

[#24001] ruby-mode.el のインデント — GOTO Naohisa <ngoto@...>

[#24006] core dump with drb in test — Tanaka Akira <akr@...17n.org>

[ruby-dev:23893] Re: cgi, cgi/session

Thread

In This Thread

[#23876] cgi, cgi/session — 堀川久 <vzw00011@...>