search

nekochans / lgtm-cat-frontend

lgtm-cat（サービス名 LGTMeow https://lgtmeow.com) のフロントエンド用
https://lgtmeow.com
21 stars 2 forks source link

API用のBFFはEdge Runtime利用のRoute Handlersを利用するように変更 #311

Closed keitakn closed 4 months ago

keitakn commented 5 months ago

issueURL

https://github.com/nekochans/lgtm-cat-frontend/issues/310

関連 URL

https://lgtm-cat-frontend-git-feature-issue310change-a-5750bf-nekochans.vercel.app/

このPRで対応すること / このPRで対応しないこと

https://github.com/nekochans/lgtm-cat-bff の利用を停止してBFF層をRoute HandlersのEdge Runtimeを利用するように変更する。

Storybook の URL もしくはスクリーンショット

UI変更はないのでなし

変更点概要

https://github.com/nekochans/lgtm-cat-bff の利用を停止してBFF層をRoute HandlersのEdge Runtimeを利用するように変更。

不要になった環境変数を削除して必要な環境変数を追加している。

なおCognitoのAccessTokenは1時間に一回だけ発行すれば良いので発行したTokenはupstashのRedisにcacheさせるようにしている。

レビュアーに重点的にチェックして欲しい点

特になし

補足情報

特になし

vercel[bot] commented 5 months ago

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
lgtm-cat-frontend ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 26, 2024 5:32am
codecov[bot] commented 5 months ago

Codecov Report

Attention: Patch coverage is 24.06816% with 713 lines in your changes are missing coverage. Please review.

Project coverage is 21.84%. Comparing base (d775e05) to head (0bb2413).

Files Patch % Lines
src/app/api/lgtm-images/route.ts 0.00% 190 Missing and 1 partial :warning:
src/app/api/cat-images/validation-results/route.ts 0.00% 122 Missing and 1 partial :warning:
src/api/cognito/openIdConnect.ts 23.77% 93 Missing :warning:
src/app/api/lgtm-images/recently-created/route.ts 0.00% 83 Missing and 1 partial :warning:
src/features/validator.ts 30.33% 62 Missing :warning:
src/middleware.ts 0.00% 24 Missing :warning:
src/app/api/_utils/http.ts 0.00% 21 Missing and 1 partial :warning:
src/templates/TopTemplate/TopTemplate.stories.tsx 0.00% 15 Missing :warning:
src/edge/url.ts 0.00% 12 Missing and 1 partial :warning:
src/constants/upstash.ts 14.28% 12 Missing :warning:
... and 11 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #311 +/- ## ========================================== - Coverage 22.39% 21.84% -0.55% ========================================== Files 229 242 +13 Lines 8936 9740 +804 Branches 304 302 -2 ========================================== + Hits 2001 2128 +127 - Misses 6761 7432 +671 - Partials 174 180 +6 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.