読者です 読者をやめる 読者になる 読者になる

なるようになるブログ

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

rails commit log流し読み(2017/03/11)

2017/03/11分のコミットです。

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

actionpack/CHANGELOG.md


Make sure local cache cleared even it’s throwing:

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

ActiveSupport::Cache::Strategy::LocalCache moduleでエラー発生時のcache clear処理にrescue Exceptionを使用していたのを、ensureを使用して処理を行うよう修正しています。

+            cleanup_on_body_close = true
             response
           rescue Rack::Utils::InvalidParameterError
-            LocalCacheRegistry.set_cache_for(local_cache_key, nil)
             [400, {}, []]
-          rescue Exception
-            LocalCacheRegistry.set_cache_for(local_cache_key, nil)
-            raise
+          ensure
+            LocalCacheRegistry.set_cache_for(local_cache_key, nil) unless
+              cleanup_on_body_close
           end

wardenを使用している場合に、wardenがthrowされたエラーをcatchしてしまい、LocalCacheの後処理が行われないことがある為、確実に処理が行われるようにする為にensureを使用するようにしたとのことです。


Improve readability of testing guide [ci skip]

rails guideのA Guide to Testing Rails Applicationsの修正です。

Rails meets Minitestの項、testメソッドについて説明している箇所の言い回しを修正しています。


Merge pull request #28355 from Edouard-chin/reversemerge-for-strongparameters

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

ActionController::Parametersクラスにreverse_merge及びreverse_merge!メソッドを追加しています。


Add @takkanm credits in the changelog [ci skip]

actionpack/CHANGELOG.mdの修正です。

先のreverse_mergereverse_merge!メソッドの対応についてのエントリーのクレジットにtakkanm さんを追加しています。元々同じ内容のPRをtakkanmさんが出されていた為。


Fix indentation and formatting in CHANGELOGs [ci skip]

actionpack/CHANGELOG.mdactivesupport/CHANGELOG.mdの修正です。

各エントリーのフォーマット、インデントの修正を行っています。


[docs] remove unnecessary sentence

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

ActionController::APIクラスのdoc内に不要な説明があったのを削除しています。