なるようになるブログ

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

rails commit log流し読み(2016/05/24)

2016/05/24分のコミットです。

CHANGELOGへの追加はありませんでした。


Respect log_warning_on_csrf_failure setting for all CSRF failures

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

異なるオリジンからJSのリクエストが行われた際に、その旨ワーニングのログを出力するようになっているのですが、log_warning_on_csrf_failurefalseを設定されている場合は、そのログを出力しないよう修正しています。


Add default exceptions affected by suppress (#25099)

activesupport/lib/active_support/core_ext/kernel/reporting.rbの修正です。

Kernel#suppressメソッドに引数(無視するexception classes)を指定しない場合に、デフォルトでStandardErrorを無視するよう修正しています。

が、suppressに引数を指定しない、という使い方はそもそもおかしいのでは、という事で後ほどrevertされています。


Revert "Add default exceptions affected by suppress (#25099)"

先ほどのKernel#suppressの対応をrevertしています。


Use RAILS_MAX_THREADS as pool size on all adapters

railtiesの修正です。

各adapter用のconfigファイルで、デフォルトのpool sizeにRAILS_MAX_THREADSの値を使用するよう修正しています。

例。

# railties/lib/rails/generators/rails/app/templates/config/databases/mysql.yml
-  pool: 5
+  pool: <%%= ENV.fetch("RAILS_MAX_THREADS") { 5 } %>

Pumaのスレッド数(threads_count)とpool sizeを合わせる為、RAILS_MAX_THREADSを使用するように修正しています。


[] and read_attribute are not aliases [ci skip]

activerecord/lib/active_record/attribute_methods.rbのdocの修正です。

ActiveRecord::AttributeMethods#[]メソッドのdocに、[]メソッドread_attributeメソッドのaliasである旨説明があったのですが、実際はaliasでは無い為、該当の記述を削除しています。