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

convUriTopath(), getErrorPage() system call함수 error시 ServerError로 대체 필요 #52

Open
Jake1152 opened this issue Apr 11, 2023 · 2 comments
Assignees

Comments

@Jake1152
Copy link
Member

Jake1152 commented Apr 11, 2023

내부에서 파일 존재 종류 확인하는 파일의 stat()함수에서 발생하는 에러 확인하면 될듯

@Jake1152
Copy link
Member Author

Jake1152 commented Apr 11, 2023

  • convUriToPath()랑 getErrorPage()에서 쓰는
    getFileExistanceType()
    함수에서 stat() 시스템콜 함수를 사용.

  • void ServerHandler::throwServerError(std::string msg, ClientSocketData* const & client_socket)
    쓸까하고 보니까 파라미터들이 convUriToPath()랑 getErrorPage()에서는 없어서 쓸수 없음

  • 처리를 한다면 getFileExistanceType() return 값으로 새로운 enum을 추가해서 convUriToPath()랑 getErrorPage()를 호출한 쪽에서 throwServerError()를 써서 처리하면 될 것으로 생각됨
    확인해보니 convUriToPath, getErrorPage 두 함수를 호출한 함수에서는 client_socket값을 가지고 있어서 throwServerError() 호출 가능할 것으로 예상

@Jake1152 Jake1152 self-assigned this Apr 12, 2023
@Jake1152
Copy link
Member Author

getFileExistanceType() 파라미터로 ClientSocketData* const & client_socket 를 추가해서 stat()에서 에러발생시 바로 error 를 보내게 만듦
getFileExistanceType()호출하는 convUriToPath(), getErrorPage()와
convUriToPath(), getErrorPage() 를 호출하는 쪽에서 client_socket을 인자로 넣어준다.

Configinfo에서 ServerHandler.hpp를 include를 해야하는 부분이 결합도가 높아진다고 생각들어서 추후에
리팩터링이 필요할지는 논의 필요

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

No branches or pull requests

1 participant