Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the OAuth2 frontend redirect URI from a local development address to a production Netlify URL. The review feedback suggests using an environment variable with a default value instead of hardcoding the production URL, which allows for better flexibility across different environments and prevents the need for manual code changes during local development.
|
|
||
| # Frontend | ||
| app.oauth2.front-redirect-uri=http://localhost:5173/oauth/callback | ||
| app.oauth2.front-redirect-uri=https://z-igma.netlify.app/oauth/callback |
There was a problem hiding this comment.
์ด์ ํ๊ฒฝ์ ์ฃผ์๋ฅผ ์ค์ ํ์ผ์ ์ง์ ๊ธฐ๋กํ๋ฉด ๋ก์ปฌ ๊ฐ๋ฐ ํ๊ฒฝ์์ ๋งค๋ฒ ์ฝ๋๋ฅผ ์์ ํด์ผ ํ๋ ๋ฒ๊ฑฐ๋ก์์ด ๋ฐ์ํฉ๋๋ค. ํ๋ก์ ํธ ๋ด ๋ค๋ฅธ ์ค์ (JWT ๋ฑ)๊ณผ ๋์ผํ๊ฒ ํ๊ฒฝ ๋ณ์๋ฅผ ์ฌ์ฉํ๊ณ , ๊ธฐ๋ณธ๊ฐ์ผ๋ก ๋ก์ปฌ ์ฃผ์๋ฅผ ์ง์ ํ๋ ๋ฐฉ์์ ๊ถ์ฅํฉ๋๋ค. ์ด๋ ๊ฒ ํ๋ฉด ๋ฐฐํฌ ํ๊ฒฝ(Netlify ๋ฑ)์์ FRONT_REDIRECT_URI ํ๊ฒฝ ๋ณ์๋ง ์ค์ ํ์ฌ ์์ ํ๊ฒ ์ด์ ์ฃผ์๋ฅผ ์ ์ฉํ ์ ์์ต๋๋ค.
app.oauth2.front-redirect-uri=${FRONT_REDIRECT_URI:http://localhost:5173/oauth/callback}
๐ ๊ด๋ จ ์ด์
โ ์์ ๋ถ๋ฅ
โจ ์์ ๋ด์ฉ
๐ฅ ์ ๋ฌ์ฌํญ
โ ์ฒดํฌ๋ฆฌ์คํธ
๐ธ ์คํฌ๋ฆฐ์ท
๐ก ๋ฐฐ์ด ๊ฒ / ์๋ํ ๊ฒ / ๊ณ ๋ฏผํ ์