search

hummingbird-jp / stat-web

1 stars 0 forks source link

Enable AppCheck #192

Closed yhakamay closed 2 years ago

yhakamay commented 2 years ago

AppCheck is ready to protect our Firebase API key.

yhakamay commented 2 years ago

Run failed の原因調査中です…

ryutarosuda commented 2 years ago

@yhakamay これついて行けてないけど、大変そうだね。iPhoneから触ってたら間違えてReady for review にしてしまった！

yhakamay commented 2 years ago

AppCheckのサーバーから 403 Forbidden が返ってきてるのがわかったので、PRのURLを登録してみた

スクリーンショット 2021-09-21 18 10 01
yhakamay commented 2 years ago

うまくいったっぽいな… となると、AppCheck有効化後はPRのURLが使えなくなるかも…？もう少し調べてみます

スクリーンショット 2021-09-21 18 10 57
yhakamay commented 2 years ago

@RQ2L84Q-ROW 何かと障壁が高いので、一旦このPR保留で！ユーザー増えてきて不審なトラフィックを確認したら再考します

ryutarosuda commented 2 years ago

了解！ありがとう！！

github-actions[bot] commented 2 years ago

Visit the preview URL for this PR (updated for commit 7c36c94):

https://stat-web-6372a--pr192-enable-appcheck-3leronuf.web.app

(expires Tue, 28 Sep 2021 13:33:56 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

yhakamay commented 2 years ago

@RQ2L84Q-ROW いけた〜お願いします！Debug Providerを設定してないのが原因でした https://firebase.google.com/docs/app-check/web/debug-provider?hl=en#ci

ryutarosuda commented 2 years ago

@yhakamay bundle.js のコンフリクト、無視してマージしちゃって大丈夫なやつ？笑

ryutarosuda commented 2 years ago

@yhakamay コマンドラインでコンフリクト解消（というかbundle.jsを改めてコンパイル）してマージしました！

yhakamay commented 2 years ago

@RQ2L84Q-ROW ありがとー！助かります