Skip to content

Conversation

@clainyun
Copy link
Member

📌 PR 제목

fix | sprint2 | FRB-188 | MQTT 연결 관련해서 재연결 로직 추가 | 윤다인


✨ 변경 사항

  • MQTT 클라이언트 연결 재시도 로직 추가
    • 최대 5회 재시도 메커니즘 구현
    • 재시도 간 2초 대기 시간 설정
  • MQTT 연결 설정 개선
    • 클라이언트 ID 고유성 보장을 위한 타임스탬프 추가
    • Keep Alive 간격 설정 (60초)
    • 최대 인플라이트 메시지 수 제한 (100개)
    • 연결 타임아웃 30초로 증가
  • 예외 처리 및 로깅 강화
    • 연결 실패 시 상세 로그 추가
    • 구독 실패 시 예외 처리 개선
    • zoneId 유효성 검사 추가

📸 스크린샷 (선택)

변경 전 변경 후
클라이언트가 연결되지 않음 에러 발생 연결 재시도 및 상세 로그 출력

✅ 체크리스트

  • 코드에 불필요한 부분은 없는가?
    • 중복 코드 제거 및 상수화 완료
  • 기능이 정상 동작하는가?
    • 연결 실패 시 재시도 동작 확인
    • 구독 기능 정상 동작 확인
  • 의존성은 문제가 없는가?
    • 기존 MQTT 라이브러리 사용, 추가 의존성 없음
  • 커밋 메시지는 명확한가?
    • "fix: MQTT 클라이언트 연결 안정성 개선"

📎 관련 이슈

  • MQTT 클라이언트 연결 실패 문제 해결: #FRB-188

💬 추가 설명

  • MQTT 연결 실패 시 자동 재연결을 시도하도록 개선했습니다.
  • 연결 상태를 모니터링하고 실패 시 상세한 로그를 남기도록 했습니다.
  • AWS IoT Core와의 연결 안정성을 위해 Keep Alive 및 타임아웃 설정을 조정했습니다.
  • 클라이언트 ID에 타임스탬프를 추가하여 여러 인스턴스 실행 시 발생할 수 있는 충돌을 방지했습니다.
  • 추후 모니터링을 위해 주요 연결 이벤트에 대한 로깅을 강화했습니다.

리뷰어 분들께서는 특히 다음 부분을 중점적으로 봐주시면 감사하겠습니다:

  1. 재연결 시도 횟수와 간격이 적절한지
  2. 예외 처리가 충분한지
  3. 로그 레벨과 메시지가 적절한지

@clainyun clainyun requested a review from gwangbu-desu May 25, 2025 09:10
@clainyun clainyun self-assigned this May 25, 2025
Copy link
Member

@gwangbu-desu gwangbu-desu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good Job

@gwangbu-desu gwangbu-desu merged commit 9768252 into develop May 25, 2025
1 check passed
@wdd1016 wdd1016 deleted the refactor/FRB-188 branch June 3, 2025 10:54
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

Successfully merging this pull request may close these issues.

3 participants