Skip to content
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.

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

Feature/allowed hosts #33

Merged
merged 3 commits into from
Mar 10, 2021
Merged

Feature/allowed hosts #33

merged 3 commits into from
Mar 10, 2021

Conversation

pacokwon
Copy link
Member

This PR drops the PUBLIC_URL environment variable and public option in webpack config in favor of allowedHosts option in webpack.

기존에 PUBLIC_URL 환경변수의 입력이 필요했던 이유는 webpack-dev-server에서 host checking이 필요했기 때문입니다.
webpack-dev-server에서 host checking은 이 repo를 원격 서버에서 실행시킬 때 일종의 보안설정입니다.
disableHostCheck를 이용해 host checking을 아예 하지 않을 수도 있지만 개발 모드라도 credential들이 접근 가능할 수 있기 때문에 이와 같은 설정이 있었습니다.
webpack-dev-server disableHostCheck 보안 관련 이슈

그러나 개발환경을 설정할 때마다 이를 입력하는 것이 번거롭기에 사실상 사용하는 두 개의 도메인, .sparcs.orglocalhostallowedHosts로 미리 등록해 두었습니다. 이와 동시에 webpack.config.js에서는 .env 파일을 읽어올 필요가 없어졌기에 dotenv 디펜던시를 제거하였습니다.

@pacokwon pacokwon self-assigned this Mar 10, 2021
@pacokwon pacokwon merged commit ea6ae6c into develop Mar 10, 2021
@pacokwon pacokwon deleted the feature/allowed-hosts branch March 10, 2021 06:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant