Skip to content

Fix/getcloser security config #346

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weโ€™ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
soohyunme wants to merge 9 commits into
base: deploy/cert-develop
Choose a base branch
from
Open

Fix/getcloser security config #346

soohyunme wants to merge 9 commits into from

Conversation

@soohyunme
Copy link
Member

@soohyunme soohyunme commented Feb 7, 2026

๐Ÿ“ PR ํƒ€์ž…

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

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

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

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

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

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

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

  • Closes #

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

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

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

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

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

@soohyunme
Merge pull request #341 from Pseudo-Lab/docs/11th-plan
a571a33 
docs: Add 11th plan
@soohyunme
Merge pull request #343 from Pseudo-Lab/deploy/cert-develop
ffa3433 
fix: improve client IP detection for access logging
@soohyunme
feat: enhance platform access logging with ip hash and unify salt config
b199214
@soohyunme
Merge pull request #344 from Pseudo-Lab/feat/enhance-logging
443de1a 
feat: enhance platform access logging with ip hash and unify salt config
@soohyunme
chore(getcloser): enhance security config with environment variables โ€ฆ
2907e2c 
โ€ฆand validation
@soohyunme
chore(ci): add SECRET_KEY and ENVIRONMENT to getcloser deployment worโ€ฆ
0883760 
โ€ฆkflow
@soohyunme
chore(getcloser): enhance security config with environment variables โ€ฆ
81491dd 
โ€ฆand validation

* chore(getcloser): enhance security config with environment variables and validation

* chore(ci): add SECRET_KEY and ENVIRONMENT to getcloser deployment workflow
@soohyunme
chore(cert): enhance cors config with environment variables
aca24f3
@soohyunme
Merge branch 'fix/cert-cors-config' into fix/getcloser-security-config
21981ae
@soohyunme soohyunme requested a review from yesinkim February 7, 2026 20:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yesinkim yesinkim Awaiting requested review from yesinkim

Assignees

No one assigned

Labels

needs-review ์ˆ˜๋ฃŒ์ฆ

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@soohyunme