なるようになるブログ

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

rails commit log流し読み(2023/07/28)

2023/07/28分のコミットです。

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

activesupport/CHANGELOG.md

activestorage/CHANGELOG.md


Add ActionDispatch::Cookies middleware test with Rack::Lint

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

ActionDispatch::Cookies middlewareをRack::Lintでチェックするよう修正しています。ActionDispatch::Cookiesが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Add Rack::Lint to ActionDispatch::Executor tests

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

ActionDispatch::Executor middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


This fix corrects the logic in both the associated and missing methods. If the @scope.values has more than one key it will use associated in the WHERE IS NOT NULL SQL clause. This fixes 48651 which was not returning the correct value when using an enum association and querying for a record that had one key of the enum but was missing another. This also fixes 44964 where ActiveRecord was not properly aliasing self-referencing relations. If the @scope.values has only one key it will use reflection.table_name in the WHERE IS NOT NULL SQL clause. This fixes 47909 in which a user was trying to find records that were simply missing a relation which resulted in an ActiveRecord Exception.

activerecord/lib/active_record/relation/query_methods.rbの修正です。

where.missingenumを使用しているassociationを指定した場合に正しく結果が取得出来ないバグがあったのを修正しています。


Add Rack::Lint to ActionDispatch::MiddlewareStack tests

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

ActionDispatch::MiddlewareStack middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48814 from skipkayhil/hm-rack-lint-host-authorization

actionpack/lib/action_dispatch/middleware/host_authorization.rbactionpack/test/dispatch/host_authorization_test.rbの修正です。

ActionDispatch::HostAuthorization middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48811 from nunosilva800/ns-rack-lint-assume-ssl

actionpack/lib/action_dispatch/middleware/assume_ssl.rbactionpack/test/dispatch/assume_ssl_test.rbの修正です。

ActionDispatch::AssumeSSL middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Lazily deserialize cache entries

activesupport/lib/active_support/cache/coder.rbの修正です。

cache format version >= 7.1、または、custom serializerを使用している場合に、expired、または、version-mismatchedなcache entryを値のdeserializingをせずに検知出来るよう修正しています。


Merge pull request #48813 from skipkayhil/hm-rack-lint-actionable-actionable-exceptions

Action Packの修正です。

ActionDispatch::ActionableExceptions middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Add Rack::Lint to DebugLocks tests

actionpack/lib/action_dispatch/middleware/debug_locks.rbactionpack/test/dispatch/debug_locks_test.rbの修正でs.

ActionDispatch:DebugLocks middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48810 from nunosilva800/ns-rack-lint-callbacks

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

ActionDispatch::Callbacks middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48818 from skipkayhil/hm-rack-list-permissions-policy

Action Packの修正です。

ActionDispatch::PermissionsPolicy middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48825 from nunosilva800/ns-rack-lint-executor

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

ActionDispatch::Executor middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Define constants for headers in ActionDispatch::Constants to avoid conditional checks around multiple classes

actionpack/lib/action_dispatch.rbactionpack/lib/action_dispatch/constants.rbの修正です。

複数のmiddlewareで使用している同じ定数を1つのファイルで定義するよう修正しています。


Use constants defined in ActionDispatch::Constants instead of conditional assignments

Action Packの修正です。

先のコミットで定義した共通の定数を各処理で使用するよう修正しています。


Move X-Request-Id header name to ActionDispatch::Constants

actionpack/lib/action_dispatch/constants.rbactionpack/lib/action_dispatch/railtie.rbの修正です。

X-Request-Idの定義を定数に移動しています。


Merge pull request #48823 from Gambitboy/audio_analyzer_metadata_tags

activestorage/lib/active_storage/analyzer/audio_analyzer.rbの修正です。

audio analyzer metadataにtagsを含むよう修正しています。tagsにはencoderなどの情報が含まれるとの事です。


Add Rack::Lint to ContentSecurityPolicy::Middleware tests

Action Packの修正です。

ActionDispatch::ContentSecurityPolicy middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。


Merge pull request #48788 from ylecuyer/fix-selenium-tests-yle

actionpack/lib/action_dispatch/system_testing/browser.rbの修正です。

seleniumdriver_pathにStringを指定出来るよう修正しています。


Merge pull request #48819 from skipkayhil/hm-doc-integration-tweaks

actionpack/lib/action_dispatch/testing/integration.rbのdocの修正です。

ActionDispatch::Integration module配下の各クラス、メソッドのdocのフォーマットや言い回しを修正しています。


Add Rack::Lint to RequestId tests

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

ActionDispatch::RequestId middlewareをRack::Lintでチェックするよう修正しています。middlewareが Rack SPECを満たしているかどうかをCIでチェック出来るようにする為。