Closed hsbt closed 7 months ago
Describe the bug / 不具合の説明
Rack 3 (実際には rackup) の Webrick handler のインタフェースが変わっていることで tDiary が webrick を使った rackup で動かなくなってます。ruby-jp のユーザーと @shugo さんから報告があって発見しました。
Additional context
具体的には Rack 2 では以下のように rack.input が StringIO だったのに対して
rack.input
StringIO
https://github.com/rack/rack/blob/2-2-stable/lib/rack/handler/webrick.rb#L71
Rack3 (rackup) では Stream::Reader を include した独自クラスになりました。
Stream::Reader
https://github.com/rack/rackup/blob/main/lib/rackup/handler/webrick.rb#L95
その結果 rewind が動かなくなりました。
rewind
この辺のコードは rack 対応したときの名残なので、実は rewind などは不要なのではないかと思いますが次のリリースまでには直したい。
Fixed by https://github.com/tdiary/tdiary-core/pull/1148
hsbt
commented
7 months ago hsbt
commented
7 months ago
Describe the bug / 不具合の説明
Rack 3 (実際には rackup) の Webrick handler のインタフェースが変わっていることで tDiary が webrick を使った rackup で動かなくなってます。ruby-jp のユーザーと @shugo さんから報告があって発見しました。
Additional context
具体的には Rack 2 では以下のように
rack.inputがStringIOだったのに対してhttps://github.com/rack/rack/blob/2-2-stable/lib/rack/handler/webrick.rb#L71
Rack3 (rackup) では
Stream::Readerを include した独自クラスになりました。https://github.com/rack/rackup/blob/main/lib/rackup/handler/webrick.rb#L95
その結果
rewindが動かなくなりました。この辺のコードは rack 対応したときの名残なので、実は
rewindなどは不要なのではないかと思いますが次のリリースまでには直したい。