なるようになるブログ

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

rails commit log流し読み(2018/08/01)

2018/08/01分のコミットです。

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


Log the remote IP addr of clients behind a proxy

railties/lib/rails/rack/logger.rbの修正です。

Rack::Loggerが記録するログのipremote_ipに変更しています。

proxy経由している場合にも正しいIPアドレスを記録出来るようにする為。


Merge pull request #33488 from znz/avoid-to-post-twice

actionview/app/assets/javascripts/rails-ujs/start.coffeeの修正です。

rails-ujsjquery_ujsが両方ロードされてしまっていた場合に、Errorをthrowするよう修正しています。

jquery_ujsからrails-ujsに移行している際に誤って両方ロードしてしまい、ajax処理が2回実行されてしまった、という事があったらしくそれを防ぐため。


Merge pull request #31883 from bogdanvlviv/remove-label-in-porgress-for-M-V-guides

guides/source/documents.yamlの修正です。

Active Model Basics guide、及び、Action View Overview guideから"work in progress"ラベルを削除しています。

どちらのguideももう十分に情報が記載されているからWIPじゃなくて良いだろう、との事でラベルが削除されたのですが、まだ確認が必要との事で直後にrevertされています。


Revert "Remove label 'work in progress' for AM and AV guides"

という訳で直前のguideのWIP対応をrevertしています。


Merge pull request #28937 from maclover7/jm-fix-28927

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

headメソッドのデフォルトのcontent typeにtext/htmlを指定するよう修正しています。

format.any { head 404 }のようにcontent typeを指定せずにheadを使用した場合に、content typeが未指定になるのを避ける為。


Add test case for the #5153's default order contract

activerecord/test/cases/finder_test.rbの修正です。

firstがデフォルトでorderに依存することを確認するテストを追加しています。


Revert "Merge pull request #24131 from brchristian/limit_and_primary_key"

limit()が指定されている場合に、primary keyでのorderを行わないよう修正した、Merge pull request #24131 from brchristian/limit_and_primary_keyをRevertしています。

この挙動変更はbreaking changeであり、変更するなら、まず元の挙動をdeprecateにする必要がある為。


Retry ActiveStorage::PurgeJobs on DB deadlock

activestorage/app/jobs/active_storage/purge_job.rbの修正です。

blobのpurge処理でActiveRecord::Deadlockedが発生した場合、retryするよう修正しています。


use load_schema instead of migrate for parallel testing

activerecord/lib/active_record/tasks/database_tasks.rbactiverecord/lib/active_record/test_databases.rbの修正です。

parallel testingでDBのセットアップをするのにmigrateを実行していたのを、schemaファイルをloadするよう修正しています。そちらの方が高速な為。


Remove unused require

activestorage/lib/active_storage/service/gcs_service.rbの修正です。

使用していないrequireを削除しています。