なるようになるブログ

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

rails commit log流し読み(2022/01/29)

2022/01/29分のコミットです。

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


Revert #38957 (#44287)

Active Storageのdirect upload処理でupload先を引数で指定出来るよう対応した、Pass service_name param to DirectUploadsController、及び、Fixes for multi-service direct uploadsをRevertしています。

この対応では、direct upload先を保持するのに動的に生成するtokenを使用しているのですが、このtokenはviewのrender時に生成される値の為、ページを部分的に置き換えるTurboと相性が悪い、かつ、そもそも改ざんされる恐れがある以上、クライアント側でアップロード先を指定出来るようにすべきではない(サーバ側でハンドリング出来るようにすべき)のではないか、という議論になりRevertされました。この議論は Issues with new direct upload support for multiple storage services and an alternativeで続いています。