Skip to content

Fix/getcloser security config#346

Open
soohyunme wants to merge 9 commits intodeploy/cert-developfrom
fix/getcloser-security-config
Open

Fix/getcloser security config#346
soohyunme wants to merge 9 commits intodeploy/cert-developfrom
fix/getcloser-security-config

Conversation

@soohyunme
Copy link
Member

๐Ÿ“ PR ํƒ€์ž…

  • ๋ฒ„๊ทธ ์ˆ˜์ • (fix)
  • ์‹ ๊ทœ ๊ธฐ๋Šฅ (feat)
  • ๋ฌธ์„œ ์—…๋ฐ์ดํŠธ (docs)
  • ์ฝ”๋“œ ์Šคํƒ€์ผ / ํฌ๋งทํŒ… (style)
  • ๋ฆฌํŒฉํ† ๋ง (refactor)
  • ํ…Œ์ŠคํŠธ (test)
  • ๊ธฐํƒ€ (chore, ci ๋“ฑ)

๐Ÿ”จ ์ž‘์—… ๋‚ด์šฉ

  • ๋ณด์•ˆ ๊ฐ•ํ™”๋ฅผ ์œ„ํ•ด Backend์˜ CORS ์„ค์ •์„ ๋ณ€๊ฒฝํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ๊ธฐ์กด์˜ ๋ชจ๋“  ์˜ค๋ฆฌ์ง„ ํ—ˆ์šฉ(allow_origins=["*"]) ์„ค์ •์„ ์ œ๊ฑฐํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ํ™˜๊ฒฝ ๋ณ€์ˆ˜ CORS_ORIGINS๋ฅผ ํ†ตํ•ด ํ—ˆ์šฉํ•  ์˜ค๋ฆฌ์ง„์„ ์„ค์ •ํ•  ์ˆ˜ ์žˆ๋„๋ก ๋กœ์ง์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.

  • ํ™˜๊ฒฝ ๋ณ€์ˆ˜๊ฐ€ ์—†์„ ๊ฒฝ์šฐ, ๋กœ์ปฌ ๊ฐœ๋ฐœ ํ™˜๊ฒฝ ๋ฐ ์šด์˜/๊ฐœ๋ฐœ ๋„๋ฉ”์ธ์„ ๊ธฐ๋ณธ๊ฐ’์œผ๋กœ ํ—ˆ์šฉํ•˜๋„๋ก ์„ค์ •ํ–ˆ์Šต๋‹ˆ๋‹ค.

์—ฎ์ธ ์ด์Šˆ

  • Closes #

๐Ÿ“ธ ์Šคํฌ๋ฆฐ์ƒท


๐Ÿง‘โ€๐Ÿ’ป ํ…Œ์ŠคํŠธ ๊ฒฐ๊ณผ

  • ๋กœ์ปฌ ํ™˜๊ฒฝ์—์„œ CORS_ORIGINS ํ™˜๊ฒฝ ๋ณ€์ˆ˜ ์„ค์ • ์œ ๋ฌด์— ๋”ฐ๋ฅธ ์˜ค๋ฆฌ์ง„ ํ—ˆ์šฉ ๋™์ž‘ ํ™•์ธ

๐Ÿ“… ์ฒดํฌ๋ฆฌ์ŠคํŠธ

  • ์ดํ•ดํ•˜๊ธฐ ์–ด๋ ค์šด ๋ถ€๋ถ„์— ์ฃผ์„์„ ์ถ”๊ฐ€ํ–ˆ์Šต๋‹ˆ๋‹ค.
  • ๊ด€๋ จ๋œ ๋ฌธ์„œ๋ฅผ ์—…๋ฐ์ดํŠธํ–ˆ์Šต๋‹ˆ๋‹ค.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant