2018/08/01分のコミットです。
CHANGELOGへの追加はありませんでした。
Log the remote IP addr of clients behind a proxy
railties/lib/rails/rack/logger.rb
の修正です。
Rack::Logger
が記録するログのip
をremote_ip
に変更しています。
proxy経由している場合にも正しいIPアドレスを記録出来るようにする為。
Merge pull request #33488 from znz/avoid-to-post-twice
actionview/app/assets/javascripts/rails-ujs/start.coffee
の修正です。
rails-ujs
とjquery_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.rb
、
activerecord/lib/active_record/test_databases.rb
の修正です。
parallel testingでDBのセットアップをするのにmigrateを実行していたのを、schemaファイルをloadするよう修正しています。そちらの方が高速な為。
activestorage/lib/active_storage/service/gcs_service.rb
の修正です。
使用していないrequireを削除しています。