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

Config parsing 기능 수정 및 추가 고려사항 #28

Open

Jake1152 opened this issue Apr 9, 2023 · 4 comments

Member

Jake1152 commented Apr 9, 2023 •

edited

Loading

location root
root가 비어있을때 (설정되지 않았을때) server locatino에 있는 value로 할당
port중복이 있는 경우 error로할지 무시할지
현재는 처음에 있는 것만 걸리며, nginx의 경우에도 뒤에 있는건 무시함, 대신 warning을 띄워줌

Jake1152 changed the title ~~Config parsing 기능 수정 및 추가~~ Config parsing 기능 수정 및 추가 고려사항

Member Author

Jake1152 commented Apr 9, 2023 •

edited

Loading

에러처리 현재 true.false로 판별하던 것에서 throwError()로 대체 필요
bool로 return하던것 throwError()로만 되면 void로 변경 가능할수 있음

Member Author

Jake1152 commented Apr 9, 2023 •

edited

Loading

error_page처럼 인자가 3개 이상인 경우 ';'이후로도 어떤 값이 들어와도 에러 처리하지 않는 부분에 대해서 시간이 남으면 처리되도록 수정 필요

Member Author

Jake1152 commented Apr 9, 2023

index 입력 여러개 가능하게 할지

Member Author

Jake1152 commented Apr 9, 2023

location path 완전일치가 아닌 부분 일치도 가능하게 할지
길이수를 기준으로 내림차순으로 정렬한 상태로 한다면 config 부분에서 처리는 크게 걸리지 않을것으로 예상
하지만 다른 파트와 연계되어서 수정해야할 사항이 있는지 확인 필요

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