'채팅방' 이미지 처리 추가 API 개발

[cotchan]

작업내용

• 채팅방에서 '이미지' 복수 개 전송할 수 있도록 처리
• 채팅방에서 '이미지' 전송 시 sync를 맞출 수 있도록 추가 API 개발

PR

• 83

• 84

목차

1. 채팅방에서 '이미지' 전송 시 sync를 맞출 수 있도록 추가 API 개발
   • 문제상황
   • 해결방법
2. 채팅방에서 '이미지' 복수 개 전송할 수 있도록 처리
   • 문제상황
   • 해결방법

---

채팅방에서 '이미지' 전송 시 sync를 맞출 수 있도록 추가 API 개발

문제상황

• 당근마켓에서는 채팅 메시지로 최대 10개까지 이미지를 보낼 수 있음
• 현재 presigned-url을 사용하기 때문에 채팅 전송 로직 이전에 presigned-url 등록 로직 선행 필요
• 클라이언트가 올바르게 리소스를 등록하고 전송을 해야 상대편이 해당 이미지를 조회하는 시점에 S3에서 조회 가능

해결방법

• 그래서 채팅이미지 생성 API라는 추가 API를 만듦
  • 해당 API는 주어진 img.jpg 파일에 대해 이미지를 실제 저장할 presigned_url과 메시지 전송 시 사용할 image_name을 함께 리턴

request 예시

{
    "img_files": ["test1.jpg"]
}

response 예시

{
    "origin_name": "test1.jpg",
    "image_name": "15785614-e0e9-4d4f-97aa-9a09abb05a6dtest1.jpg",
    "presigned_url": "https://project-club-bucket.s3.ap-northeast-2.amazonaws.com/chat-image/15785614-e0e9-4d4f-97aa-9a09abb05a6dtest1.jpg?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Date=20220626T084527Z&X-Amz-SignedHeaders=host&X-Amz-Expires=600&X-Amz-Credential=AKIA4EAIYFA45U7IOW6L%2F20220626%2Fap-northeast-2%2Fs3%2Faws4_request&X-Amz-Signature=20efb322e9ae20ccc939afd1f9e18352266b1d449d7b123b1c16ad84f5f0c924"
}

채팅방에서 '이미지' 복수 개 전송할 수 있도록 처리

문제상황

• 기존의 메시지 스펙에 복수개의 이미지를 한 번에 보내기 위한 코드 수정이 필요
  • 기존의 일반 메시지, 좌표 메시지는 String message 하나로 처리했지만 추가적인 이미지 요청은 List<String> imgUrls 필드가 필요
  • 이미지 URL도 String message로 정의하여 하나씩 보낼 수도 있지만 해당 방식은 10개 이미지를 전송시 10번의 POST 요청이 나간다는 문제 발생
• mongodb의 경우 document 속성을 유연하게 가져갈 수 있지만 application 코드에서는 VO에 String message와 List<String> imgUrls를 모두 가지고 있어야 하므로 추가적인 코드 작업이 필요
• 일반, 좌표 메시지의 경우 String message 필드에 값이 있지만 이미지 메시지의 경우 String message값은 null이 되고 List<String> imgUrls 필드에 값이 있게 된다.

해결방법

• Optional과 생성자 팩토리 메서드를 사용하여 message 필드가 null이 될 수 있음을 명시하고 이미지 메시지와 그 외 메시지를 생성하는 로직을 분리

@Getter
@NoArgsConstructor
public class ChatMessage {

    private Long senderId;
    private MessageType messageType;
    private String message;
    private List<String> imgUrls = new ArrayList<>();
    private LocalDateTime createdAt;

    public Optional<String> getMessage() {
        return ofNullable(message);
    }

    public static ChatMessage ofMessage(Long senderId,
                                        MessageType messageType,
                                        String message) {

        checkArgument(senderId != null, "senderId 값은 필수입니다.");
        checkArgument(messageType != null, "messageType 값은 필수입니다.");
        checkArgument(isNotEmpty(message), "message 값은 필수입니다.");

        return ChatMessage.builder()
                .senderId(senderId)
                .messageType(messageType)
                .message(message)
                .imgUrls(null)
                .build();
    }

    public static ChatMessage ofImage(Long senderId, List<String> imgUrls) {
        checkArgument(senderId != null, "senderId 값은 필수입니다.");
        checkArgument(imgUrls != null, "imgUrls 값은 필수입니다.");
        checkArgument(imgUrls.size() <= 10, "message 값은 필수입니다.");

        return ChatMessage.builder()
                .senderId(senderId)
                .messageType(MessageType.IMAGE)
                .message(null)
                .imgUrls(imgUrls)
                .build();
    }
    //...
}