なるようになるブログ

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

rails commit log流し読み(2022/10/22)

2022/10/22分のコミットです。

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

activerecord/CHANGELOG.md

activestorage/CHANGELOG.md

activesupport/CHANGELOG.md


Do not trigger after_commit :destroy callback again on destroy if record previously was destroyed

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

after_commit :destroy callbackが、destroyが呼ばれる度に実行されていたのを、実際にDBからデータを削除した時のみ実行されるよう修正しています。


Fix #46244 Remove innerHTML usage to avoid self-XSS

actionpack/lib/action_dispatch/middleware/templates/routes/_table.html.erbの修正です。

Route Error PageでinnerHTMLを使っている箇所があったのを、使わないよう修正しています。innerHTMLだと、不正な入力値を指定した場合にXSSが出来てしまう為。


Merge pull request #46291 from chaadow/preview-log-subscriber

activestorage/lib/active_storage/log_subscriber.rbactivestorage/lib/active_storage/previewer.rbの修正です。

ActiveStoragepreviewメソッド実行時に、uploaddownloadと同様に、Rails.logger経由でログを出力するよう修正しています。


MemoryStore: preserve entry ttl when incrementing

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

MemoryCacheStoreで、increment/decrement実行時に、TTL全体の値を保持するよう修正しています。RedisCacheStoreと挙動をあわせる為。


PR template: note about auto-closing issue

.github/pull_request_template.mdの修正です。

issueの修正の場合、コミット本文に"Fixes #xxx(issueの番号)"を含むよう説明を追加しています。