search

LuxQuad / ozet-core-api

온라인 이력서 관리 플랫폼 esume 의 API 서버
https://api.esume.me/docs
MIT License
1 stars 0 forks source link

cors 환경분리 #5

Closed JeGwan closed 3 years ago

JeGwan commented 3 years ago

cors에 개발환경만 로컬호스트 허용하고 프로덕션때는 이주메만 하믄 안될까요!?

BartKim-J commented 3 years ago

오께이 오늘이나 내일 작업칠게여

BartKim-J commented 3 years ago

오께이 오늘이나 내일 작업칠게여

CORS 이미 등록은 해놓긴했는데 제가 Allow 한 URL을 공유를 안드려서 그런거같아여 일단 개발용 API 오픈해둔게 없어서 Container 개발용으로 하나 더 올릴지는 고민해볼게용

BartKim-J commented 3 years ago

https://github.com/LuxQuad/esume-core-api/blob/master/app/middleware/cors.py

JeGwan commented 3 years ago

리스트 객체가 아래와 같았으면 좋겠어요 

prod_origins: list =  ["https://esume.me"]
dev_origins: list =  [
    "http://localhost",
    "http://localhost:8000",
    "http://localhost:8080",
    "http://localhost:5000"
]

origins: list = dev_origins if someEnvVar is "development" else prod_origins
BartKim-J commented 3 years ago

@JeGwan Check Check https://github.com/LuxQuad/esume-core-api/pull/13