commew / timelogger-web

時間記録アプリ
https://timmew.commew.net
MIT License
2 stars 0 forks source link

Request-IdはX-Request-Idにしたい #84

Closed HAYASHI-Masayuki closed 1 year ago

HAYASHI-Masayuki commented 1 year ago

概要

デバッグのために、こちらでAPIの仕様にRequest-Idを追加していただきましたが、RailsにはX-Request-Idというヘッダを、Request-Id同様に「渡されたら、そのまま返す」、「渡されなかったら、生成して返す」という機能があるようなので、そちらをそのまま使いたいと思います。

特にほかに問題がないようであれば、X-Request-Idに変更していただけると助かります。

再現手順

なし

再現環境

なし

このバグによって引き起こされる問題

なし

スクリーンショット、エラーメッセージ

なし

補足情報

RailsのX-Request-Idについてはこの辺とかに書かれています。 Request Idについて調べた - Re: 醤油の一升瓶じゃあ戦えない

keitakn commented 1 year ago

@HAYASHI-Masayuki

承知しました、一応 Request-Id とした経緯としては RFC6648X- 付きのHeaderが非推奨とされていたからですね。

https://castor.hatenadiary.org/entry/2022/10/21/005817

とはいえバックエンドの開発リソースが取れない中で進めて頂いているのは理解していますし、プロダクトを完成させることのほうが重要度は高いので X-Request-Id に変更しようと思います!

HAYASHI-Masayuki commented 1 year ago

@keitakn ありがとうございます! が、上記URL確認してふと思い立って確認したところ、X-Request-Idは設定1つでRequest-Idに変更できそうでした。

https://github.com/rails/rails/blob/0de4017d9f42702f5b235b140d896a4f96b48b9f/actionpack/lib/action_dispatch/railtie.rb#L27

ですのでやはりこちらで修正します。お手数おかけしました。