Bug #479

1.8で当てられたcgi.rbに対するバグのパッチが適用されていない

Added by Takeyuki FUJIOKA about 7 years ago. Updated over 4 years ago.

[ruby-dev:35936]
Status:Closed
Priority:Normal
Assignee:Yui NARUSE
ruby -v: Backport:

Description

=begin
1.8で適用されたパッチの1.9用のバックポートを作成しました。
添付しておきます。
=end

cgi_backport_18.patch Magnifier (6.53 KB) Takeyuki FUJIOKA, 08/24/2008 01:20 AM

History

#1 Updated by Yui NARUSE about 7 years ago

  • Category set to lib
  • Status changed from Open to Assigned
  • Assignee set to Yui NARUSE

=begin
とりあえずいくつかコメントを。

まず総論として、適用されたパッチごとに分割してくださると助かります。
まとまっているとレビューが大変です。
1.9 に適用されていないパッチの列挙でもいいですので。

== /.../n の付与
基本的に 1.9 では不要です。(7bit 文字列の特別扱いがあるので)

== CGI::escapeHTML(string)
1.9 の gsub の第二引数に Hash を与える拡張を用いています。

== CGI::escape_html と CGI::unescape_html の削除
削除の根拠が不明です (これらは をうけた trunk における追加)

== CGI::QueryExtension::MorphingBody
1.9 での修正のようなんですが、単に 1.8 上書きしていいのか不明です。

== extend(Value)
1.8 での変更が 2003 年で古すぎるので、判断できません。

== CGI_PARAMS と CGI_COOKIES
もとの修正の意味がわからないので判断できません。
=end

#2 Updated by Takeyuki FUJIOKA about 7 years ago

=begin

== /.../n の付与
基本的に 1.9 では不要です。(7bit 文字列の特別扱いがあるので)

知りませんでした。ありがとうございます。

== CGI::escape_html と CGI::unescape_html の削除
削除の根拠が不明です (これらは をうけた trunk における追加)

すみません、いきさつ理解してませんでした。1.8のどこかで消されたのかと思いました。
確かにescape_htmlがRails流の命名規約なので、これはあったほうがいいですね。

残りは個別にまた根拠をもっと確定してからまたチケットを切りなおします。
チケットを閉じてくださって結構です。
=end

#3 Updated by Yui NARUSE about 7 years ago

  • Status changed from Assigned to Closed
  • % Done changed from 0 to 100

=begin
Applied in changeset r18801.
=end

Also available in: Atom PDF