search

fjordllc / bootcamp

プログラマー向けEラーニングシステム
https://bootcamp.fjord.jp
MIT License
281 stars 71 forks source link

【メンター向け】「はじめての日報です！」の通知メールのリンクがおかしい #6963

Closed JunichiIto closed 10 months ago

JunichiIto commented 11 months ago

概要

「○○さんのはじめての日報です！」の通知メールにある「この日報へ」ボタンのURLが http:///reports/82102 のようになっていて、クリックしても目的のページに遷移できない。（原因はなんとなく予想できるけど・・・w）

再現手順

（省略）

期待される振る舞い

該当の日報に遷移できる

スクリーンショット

fbc
natsuto6 commented 11 months ago

@JunichiIto（ @komagata @machida ） いつもお世話になっております。 こちらのIssueについて以下確認させていただきたいです。

discriptionとの相違点

  1. href="/reports/1066585819のような相対URLになっている
  2. 「日報へ」を押下すると、該当の日報に正しく遷移できる https://gyazo.com/3a35f6cbfacae34cc18867b441dd9eea

ローカル動作確認手順

  1. foreman start -f Procfile.devでローカル環境を立ち上げる
  2. nippounashiでログイン
  3. http://localhost:3000/reports/newで初めての日報を投稿
  4. ログアウト
  5. komagataでログイン
  6. 「通知」に「nippounashiさんがはじめての日報を書きました！」が存在するか確認する
  7. http://localhost:3000/letter_opener/にアクセスし、メールが送られていることを確認する
  8. 検証ツールで「この日報へ」ボタンのリンクを確認し遷移する

長くなりました。お手数ですがご確認よろしくお願い致します🙇‍♂️

machida commented 11 months ago

@natsuto6

メールの中のリンク先はどこかのサイト内ではないので、パスではなく、https:// から始まるURLにする必要がありますが、現状はそうなっていないため、本番ではアクセスできないで状況です。

なぜ、ローカルではアクセスができているかは、ローカルではletter_opener_webというメールを確認するためのGemがあり、それを使ってブラウザからメールの内容を確認しているためです。リンク先がパスであっても、メール確認のためのブラウザの画面からのパスで解釈されています。

なぜ、/xxxx と指定しているものがブラウザでは、https://xxxx となるかは、メール内ではパスでリンク先は指定できないので、リンク先は /xxxx であることは間違いなため、https://xxxx と間違いを修正してくれているからです。

JunichiIto commented 11 months ago

原因は町田さんが説明されているとおりです。

「あるあるバグ」なので、もしかすると他にも同様のバグを抱えたメールがあるかもしれません。

natsuto6 commented 11 months ago

@machida @JunichiIto ご連携いただきありがとうございます！ いただいた内容を元に対応を進めていきます🙇‍♂️

natsuto6 commented 10 months ago

本番環境で動作確認いただいたのでcloseします

JunichiIto commented 10 months ago

@natsuto6 対応ありがとうございました！