404-DriverNotFound / 200-OK

jikang, yunslee, ykoh project for Pass this project
2 stars 1 forks source link

[Fix] 파일이 존재하지 않을 때, 404 NOT FOUND가 아닌 error.html로 에러가 대체됨 #51

Closed exgs closed 3 years ago

exgs commented 3 years ago

문제점 파일이 존재하지 않을 때, 404 NOT FOUND html 파일이 뜨게 하고 싶으나, 현재 코드에서는 파일이 존재하지 않기 때문에, ft::access()에서 false를 반환하고 그 이후에는 디렉토리인지, 파일인지 알 수 있는 방법이 없어서, error.html 페이지가 뜨는 것으로 대체되고 있음.

예상되는(제대로 된) 동작 nginx를 똑같이 따라한다면, nginx의 동작을 다시 한번 확인해야함. 우리만의 룰을 만들어서 적용하는거면, 일관성만 가지고 처리하면 될 듯.

버그 재현 방법(버그를 발생시키는 방법) 프로그램을 돌리는데에 있어서는 문제가 되고있지않기 때문에 보류한다.

exgs commented 3 years ago

그냥 일관성 있게 처리하는 것이 나을 것 같음. nginx를 그대로 따라가면, 조건문에 따른 분기가 매우 많아짐.

exgs commented 3 years ago

ft::access 함수 만들어서 사용함으로써 404 NOT FOUND 처리를 올바르하게 함