Sentry を導入 #608
Sentry を導入 #608
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
HosokawaR
force-pushed
the
feature/sentry
branch
from
59ae67e
to
b520e4a
Compare
I used @sentry/rollup-plugin instead of @sentry/vite-plugin, because @senty/vite-plugin occurd building error.
HosokawaR
force-pushed
the
feature/sentry
branch
from
b520e4a
to
18e0c0d
Compare
HosokawaR
force-pushed
the
feature/sentry
branch
from
18e0c0d
to
cad3588
Compare
HosokawaR
force-pushed
the
feature/sentry
branch
from
cad3588
to
74e1260
Compare
| app.config.errorHandler = (err, _vm, _info) => { | ||
| Sentry.captureException(err); // 手動でSentryに送信 | ||
| }; |
There was a problem hiding this comment.
これは不要なため消しました。
@sentry/vue の実装を見れば分かるように、Sentry.initは自動でapp.config.errorHandlerにエラーキャプチャを仕込むためです。
逆に
window.addEventListener("error", (event) => {});
window.addEventListener("unhandledrejection", (event) => {});
などはキャプチャしてくれなそうなので、残しました。
| } else if (error instanceof InternalServerError) { | ||
| errorMessage.value = | ||
| "申し訳ございません。サーバー内でエラーが発生しました。"; | ||
| return true; | ||
| } else { | ||
| errorMessage.value = error.message; | ||
| return true; | ||
| } | ||
|
|
||
| return false; |
There was a problem hiding this comment.
https://vuejs.org/api/composition-api-lifecycle.html#onerrorcaptured にあるように、onerrorcapturedの返り値が true だと app.config.errorHandler までエラーが伝播し Sentry に補足されます。false だと補足されません。
未認証エラー、ネットワークエラーは Sentry では補足しないつもりです。（意見募集中）
| @@ -1,4 +1,5 @@ | |||
| import path from "path"; | |||
| import sentryRollupPlugin from "@sentry/rollup-plugin"; | |||
There was a problem hiding this comment.
実は @sentry/vite-plugin といのもあるのですが、こちらはビルド時に Vite がエラーを吐いてしまうため使いませんでした。
エラーが出る原因は色々調べたのですがわかりませんでした。
代わりに、 Vite はプロダクションビルド時に内部で Rollup.js を使用しており、Rollup.js の設定を vite.config.js 経由で渡せるため、@sentry/rollup-plugin を使用しました。
また、まだパッケージが公開されて浅いですが、使用することに決めました。
このプラグインは基本的にビルド時に Source Map を Sentry に送るだけなので、プロダクションに動作には大きな影響を与えないと判断したためです。
|
プロダクションデプロイ時に環境変数を渡す箇所について見てもらえると幸いです。 |
| @@ -7,7 +7,9 @@ RUN yarn install | |||
| ENV VITE_API_URL=https://app.twinte.net/api/v3 | |||
|
|
|||
| COPY . ./ | |||
| RUN yarn build | |||
| RUN --mount=type=secret,id=SENTRY_AUTH_TOKEN \ | |||
There was a problem hiding this comment.
最近 --mount=type=secret 使ってないので確認したいんですが、 # syntax= から始まる行を Dockerfile の先頭に入れてフロントエンド指定しなくてもこれって動くんでしたっけ?
There was a problem hiding this comment.
ありがとうございます。返信が遅れすみません。
https://docs.docker.com/engine/release-notes/#builder-5
Support for RUN --mount options without needing to specify experimental dockerfile #syntax directive.
入れなくても動くようになったようです。
ただ、そう言われるとこれがdocker/build-push-action@v2にでも動くかどうが自信がなくなってきました。
There was a problem hiding this comment.
上記について調べました。
Docker の 20.10.0 で # syntac=が不要になり、そしてdocker/build-push-action@v2の GitHub Actions のログを見てみると Version: 20.10.21+azure-1の記載があるので、動くと判断しました。
背景
Resolves #593
概要
細かい意図は下のコメントに続けて書きます。
なお本 PR が Merge された段階で開発メンバーに Sentry プロジェクトの招待を送ります。
動作検証
設定をいじり、ローカルで発生したエラーが Sentry に補足され、 Discord のチャンネルに流れることを確認した。