なるようになるブログ

読書感想文かrailsについてかrubyについてか

rails commit log流し読み(2023/01/21)

2023/01/21分のコミットです。

CHANGELOGにのったコミットは以下の通りです。

actionmailbox/CHANGELOG.md


Merge pull request #47066 from ioquatix/rack-3-cookies

Action Packのテストの修正です。

cookieの値をチェックするテストで、期待している値を厳密にパースに、cookieのkey/valueが実際に一致しているかをチェックするよう修正しています。Rack 3と2でcookieの生成ルールが変わっており、その影響でテストがfailするのを避けるようにする為。


Merge pull request #47056 from Moiz1524/add-libvips-note

rails guideのAction Text Overviewの修正です。

content fieldに添付ファイルが指定された場合、libvips42がインストールされていないとそれが正しく表示されない旨注記を追加しています。


Copy-fix the last PR [ci skip]

rails guideのAction Text Overviewの修正です。

先に追加された注記のグラマーの修正を行っています。


Merge pull request #47063 from codergeek121/add-rails-health-controller-test

railties/test/rails_health_controller_test.rbの修正です。

Rails::HealthControllerのテストを追加しています。


Improve matching of headers assertions.

Action Packのテストの修正です。

headerのkeyが大文字小文字どちらの場合でもテストが通るよう修正、及び、headerのチェック処理を専用のassertionに切り出して、そちらを使用するよう修正しています。


Explicitly require rack/server as it may not be part of rack.

railties/lib/rails/commands/server/server_command.rbの修正です。

不足してたrack/serverのrequireを追加しています。


Rack deprecated Rack::File and replaced it with Rack::Files.

actionpack/lib/action_dispatch/middleware/static.rbの修正です。

deprecatedになったRack::Fileの代わりにRack::Filesを使用するよう修正しています。


Don't assume Rack input supports rewindability.

actionpack/test/dispatch/request_test.rbの修正です。

Rackのinput bodyがrewindableである事を期待しているテストを、Rack 2以下でのみ実行するよう修正しています。Rack 3ではinput bodyがrewindableである事の保証はしなくなる為、とのことです。


Fix #inspect failures when dealing with requests with method=nil.

actionpack/lib/action_dispatch/http/request.rbの修正です。

HTTP method名がnilになっていた場合に、ActionDispatch::Requestインスタンスに対する#inspect処理がエラーになってしまっていたのを、エラーにならないよう修正しています。


Don't assume the existance of add_cookie_to_header and related methods.

actionpack/lib/action_dispatch/middleware/cookies.rbの修正です。

Rack 3でdeprecatedになったadd_cookie_to_header/add_remove_cookie_to_header等のメソッドを使用しないよう修正しています。


Avoid redefining the response=.

actionpack/lib/action_controller/metal.rbの修正です。

response=メソッドを複数回定義するようになっていたのを修正しています。


Merge pull request #47082 from ioquatix/rack-3-env-fields

actionpack/test/dispatch/test_request_test.rbの修正です。

Rackが設定するenvをテストでチェックしていたのをしないよう修正しています。Rack 3で必須項目ではなくなる為。


Merge pull request #47083 from ioquatix/rack-3-ssl_test-headers

actionpack/test/dispatch/ssl_test.rbの修正です。

Rack 3でdeprecatedになったRack::Utils::HeaderHashを使用しないよう修正しています。


Merge pull request #47081 from ioquatix/rack-3-query-parser-utf8

actionpack/test/dispatch/request_test.rbの修正です。

QUERY_STRINGUTF-8として不正な値を指定した場合にエラーになる事を確認するテストをRack 3以上では実行しないよう修正しています。Rack 3ではエラーにならなくなった(値をASCII-8BITとして扱うようになった)為。


Add .to_i when reading ENV[WEB_CONCURRENCY]

railties/lib/rails/generators/rails/app/templates/config/puma.rb.ttの修正です。

pumaのworker_countを設定する際に、ENV["WEB_CONCURRENCY"]の値をto_iしてから使用するよう修正しています。が、もともとIntegerを使って変換するようになっており、to_iの呼び出しは不要だった為後ほどRevertされています。


Revert "Merge pull request #47089 from mikker/patch-1"

直前のworker_countの設定処理の修正をRevertしています。


Merge pull request #47028 from bensheldon/connection_error_postgres

activerecord/lib/active_record/connection_adapters/postgresql_adapter.rbの修正です。

database名にpostgresという名前を指定、かつ、DBへの接続に失敗した場合に、ActiveRecord::NoDatabaseErrorではなくActiveRecord::ConnectionNotEstablishedがraiseされるよう修正しています。ActiveRecord::NoDatabaseErrorだとDBへの接続に失敗しているが指定されたDB名が存在しない、と本来発生しているエラーと違うエラーが発生していると勘違いしてしまう可能性がある為。


Merge pull request #46300 from honzasterba/honzasterba/mailbox_ingress_encoding_fix

Action Mailboxの修正です。

ingress controllerにbodyがUTF-8以外のencodeのデータが送信された場合にエラーになってしまっていたのを、それらのメールを受信出来るよう修正しています。


Prefix exceptions with namespace and code format

rails guideのRails Internationalization (I18n) APIの修正です。

Using Different Exception Handlersの項にあるI18n APIが使用するexceptionの一覧のexception classをnamespace付きに修正、及び、説明部分のフォーマットの修正を行っています。


Revert "Use eval_gemfile to read .Gemfile"

ローカルの.GemfileをロードするのにBundlerのeval_gemfileメソッドを使用するよう修正した、Use eval_gemfile to read .GemfileをRevertしています。eval_gemfileは現在Bundlerのpublic APIではなく、bundler以外で使用するのは推奨されていない為。


replace path argument to route helper with post object

rails guidenのWorking with JavaScript in Railsの修正です。

Turbo Framesの項にあるexampleコード内のタイポを修正しています。