search

cloudnativedaysjp / dreamkast-ui

MIT License
7 stars 2 forks source link

Add Sentry #196

Closed inductor closed 2 years ago

inductor commented 2 years ago

Sentryの導入 https://github.com/cloudnativedaysjp/dreamkast-ui/issues/149

ついでにNextが10.0.0.8以上じゃないとだめとのことだったのでバージョン固定化＋アップデート。 最新が12なので上げられるなら上げておいたほうがよさそうですが別スコープ。

dreamkast-cloudnativedays commented 2 years ago

Review app

https://dreamkast-ui-196.dev.cloudnativedays.jp

inductor commented 2 years ago

https://docs.sentry.io/platforms/javascript/guides/nextjs/manual-setup/#use-configuration-files

にある通り、

defaults.url=https://sentry.io/
defaults.org=cloudnative-days
defaults.project=dreamkast-staging

と

[auth]
token=

に対しては

Property name Environment variable
defaults.url SENTRY_URL
defaults.org SENTRY_ORG
defaults.project SENTRY_PROJECT
auth.token SENTRY_AUTH_TOKEN

という対応があるっぽい。

inductor commented 2 years ago

そもそもSentryで「Project」って使い分けてたっけ？という疑問が浮かんだ。Environmentで分けてた記憶はあるけどどうだったかな・・・

inductor commented 2 years ago

https://github.com/search?q=82e52bc9814f478a917a9f4b2a190202&type=code

DSNを見る感じ分かれてそう！！ ってことはちゃんと環境変数にしたほうがいいね