search

ztkmkoo / dss

Asynchronous distributed server system (dss)
MIT License
19 stars 13 forks source link

Origin/feature/49 multipart #125

Open pocketmun1216 opened 4 years ago

pocketmun1216 commented 4 years ago

multipart 처리할 수 있게 하였습니다.

리뷰하실 때

  1. 테스팅 코드 DssRestServerTest <- 테스팅 코드 추가

  2. Header 파싱 방법 제안 DssRestRequest <- header의 charset과 boundary 파싱하는 코드를 추가하였습니다.

  3. Body 파싱 방법 제안 DssRestActorFormDataService <- body 부분의 boundary 기준으로 파싱하는 코드를 추가하였습니다.

  4. DssRestActorService 기존 DssRestServiceResponse handling(String content) -> DssRestServiceResponse handling(DssRestServiceActorCommandRequest commandRequest) 로 변경되었습니다. 인터페이스가 변경됨에 따라 인터페이스를 구현하는 하위 클래스 또한 모두 변경되었습니다. 변경 한 이유는 DssRestActorFormDataService <- 해당 클래스에서 content 파싱시 formdata형식은 boundary로 파싱해야 하는데 기존에 content만 넘길 경우 boundary를 넘겨받기가 어렵기 때문에 헤더정보를 같이 넘겨주는 DssRestServiceActorCommandRequest 로 변경하였습니다.

  5. 실행 예제 포스트맨으로 테스팅 해보고 싶은 분들은 MultiPart 클래스 <- 참고하시면 됩니다.

ztkmkoo commented 4 years ago

PR 감사합니다.

우선 conflict 해결 및 IDE 파일 .iml은 제거 해주시면 감사하겠습니다.

ztkmkoo commented 4 years ago

PR 감사합니다.

우선 conflict 해결 및 IDE 파일 .iml은 제거 해주시면 감사하겠습니다.

@MunSeongUk 44f7104 요기 커밋에서 리뷰 반영이라고 되어 있는데, conflict가 해결 안된것 같은데 이건 아직 수정 안하신거죠?

coveralls commented 4 years ago

Coverage Status

Coverage decreased (-3.2%) to 82.196% when pulling 67aad88e68d29ba23dbffe73761cf4da3432a24d on MunSeongUk:origin/feature/49_multipart into 51068998bb36aba7b07ebe298785f6cfcb3f5afa on ztkmkoo:develop.