ErrorCode 개편

[jaelyangChoi]

📌 기능 이름

ErrorCode 체계 개편

📝 기능 설명

• ErrorCode를 매번 정의해야하고, 기존 에러로그를 수용하지 못해 관리 불편
• 비슷한 오류일 경우 같은 ErrorCode를 사용면서, 기존 에러로그를 받아 로그를 남길 수 있게 변경

✅ 기대 동작

No response

🧩 세부 작업(Task)

No response

💬 추가 정보

No response

[leeedohyun]

@jaelyangChoi 혹시 어떻게 개편하실건지 여쭤봐도 되나요? 말씀하신 ErrorCode가 현재 저희 ErrorType을 말씀하시는 걸까요?

에러코드를 매번 만들어야해서 너무 귀찮고 충돌도 잦아서요
예를들어 s3 연동 오류라고 해도 원인이 최소 3개 이상인데 이걸 에러코드를 각각 다 만드는게 아니라
Catch(Exception e) {
Throw new S3_CONN_ERR(e)
} 이런식으로 같은 타입으로 묶어 써도 에러로그는 다르게 찍히도록 하면 좋을 것 같아요

지금 에러코드 만들기 귀찮아서 대충 하나로 퉁쳤더니 에러원인 파악이 잘 안되네요 로그도 안남기고 있고. 특히 jwt나 인프라쪽은 에러 원인이 다양합니다..파악도 어렵고

[leeedohyun]

제 질문을 수정하신 것 같네요 ㅋㅋㅋ

[leeedohyun]

외부 시스템에 대한 에러 코드는 하나로 통일 해도 괜찮다고 생각합니다. 이건 클라이언트가 몰라도 되는 부분이니까요. 그 외에 비즈니스 로직에 대한 에러는 클라이언트가 구체적으로 알면 좋으니까 비즈니스 로직에는 에러 코드 작성하는 거는 동의하시는 부분일까요?

+) S3 같은 경우에는 별도의 모듈로 분리되어 있어서 예외를 catch할 때 에러 로그를 남겨도 괜찮을 것 같다는 생각이 드네요!

[jaelyangChoi]

s3와 더불어 jwt 관련해서도 에러 로그 다 남겨야 합니다. 비즈니스에 대한 부분도 아니고 원인을 파악하기 힘들어요.

에러코드를 자체를 반대하는건 아닙니다만.. 서버 로그를 보면 에러 원인 찾기가 좀 힘들더라고요,
에러코드 자체는 서버나 클라이언트에서 구체적인 문제를 잡기가 어려운 것 같아요.
제대로 쓸라면 구체적으로 케이스마다 다 작성해야 하는데, 전부 예상도 안가고 에러코드를 만드는 과정 자체가 번거롭다보니까 구체적인 정의를 안하게 되는거 같기도합니다. 결국 뭉뚱그려 표현하고..

사실상 CoreException만 던져서 controller advice에서 잡다보니까, coreException으로 정의하지 않은 example error를 더 많이 보는거 같아요; 저번에 즐겨찾기 도메인 삭제를 안해서 오류 났을때도 그렇고, 미처 예상하지 못한 에러에 대한 대응이 부족하지 않나 싶어서 좀 생각해봐야할 것 같습니다.

[leeedohyun]

어떤 이유인지 충분히 이해가 갑니다. 실제 운영 중 S3나 JWT 관련 에러는 원인 파악이 중요한 부분이라, 로그가 빠지면 문제 분석이 어렵다는 점에 저도 공감해요.
같은 생각인지는 모르겠는데, 비즈니스 로직에 직접 연관된 오류는 ErrorCode를 구체화하는 것이 분명히 의미 있지만, S3나 JWT처럼 외부 인프라에 가까운 부분은 ErrorCode를 무리하게 세분화하기보단 예외 자체를 로그로 충분히 남기는 방식이 더 현실적이지 않을까 하는 생각이에요. (물론 로그가 잘 남는다는 전제 하에요.)

그래서 아래처럼 로그를 남기는 방식을 위에서 말씀드린겁니다!

try {
    s3Client.putObject(request, RequestBody.fromBytes(file.getBytes()));
} catch (IOException e) {
    log.error("S3 업로드 중 IO 예외 발생", e);
    throw new UploadException(ErrorCode.S3_UPLOAD_FAILED_IO, e);
} catch (SdkException e) {
    log.error("S3 업로드 중 SDK 예외 발생", e);
    throw new UploadException(ErrorCode.S3_UPLOAD_FAILED_SDK, e);
}

이런 방식이면 ErrorCode가 다소 추상적이어도, 로그에 예외 정보가 잘 포함되어 있어 디버깅은 무리 없이 가능하지 않을까요?

조만간 이 부분에 대해 같이 정리해서 방향 맞춰보면 좋을 것 같아요. 시간 될 때 이야기 나눠봐요!