Skip to content

[FIX] Soniox 전사 실패 로그에 오디오 크기·형식 추가 - #287

Merged
coli-geonwoo merged 1 commit into
developfrom
fix/#286-soniox-failure-log
Oct 24, 2026
Merged

coli-geonwoo merged 1 commit into
developfrom
fix/#286-soniox-failure-log

Conversation

@coli-geonwoo

Copy link
Copy Markdown
Contributor

🚩 연관 이슈

closed #286

작업 내용

dev에서 Soniox 전사가 Invalid audio file: Error analyzing the file. The file may be corrupted or uses an unsupported format.로 실패했지만(10/24 01:45, 10건 중 1건), 로그에 업로드한 오디오 정보가 없어 어떤 파일이 문제였는지 바로 확인할 수 없었습니다.

검증

  • 실패 시 로그에 파일 이름·형식·크기가 남는지 확인하는 테스트를 먼저 실패시킨 뒤 구현했습니다.
  • ./gradlew test: 652개 통과

🗣️ 리뷰 요구사항 (선택)

  • 동시 실행 한도 초과(429)처럼 클라이언트 오류는 로그를 남기지 않습니다. 서버 오류만 남기는 범위가 적절한지 봐 주세요.

🤖 Generated with Claude Code

Soniox가 오디오를 거절하면 어떤 파일이었는지 알 수 있도록 파일 이름, 형식, 크기를 경고 로그에 남긴다.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coli-geonwoo coli-geonwoo added the fix 기능 수정 label Oct 23, 2026
@coli-geonwoo coli-geonwoo self-assigned this Oct 23, 2026
@coderabbitai

coderabbitai Bot commented Oct 23, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 18 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
📥 Commits

Reviewing files that changed from the base of the PR and between d807b49 and 94aef57.

📒 Files selected for processing: 2
  • src/main/java/com/debatetimer/service/transcript/SpeechTranscribeService.java
  • src/test/java/com/debatetimer/service/transcript/SpeechTranscribeServiceTest.java
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

Copy link
Copy Markdown

Test Results

255 files  255 suites   49s ⏱️
636 tests 636 ✅ 0 💤 0 ❌
652 runs  652 ✅ 0 💤 0 ❌

Results for commit 94aef57.

@github-actions

Copy link
Copy Markdown

📝 Test Coverage Report

Overall Project 92.66% 🍏
Files changed 100% 🍏

File Coverage
SpeechTranscribeService.java 100% 🍏

@coli-geonwoo
coli-geonwoo merged commit a41a8e4 into develop Oct 24, 2026
5 checks passed
@coli-geonwoo
coli-geonwoo deleted the fix/#286-soniox-failure-log branch October 24, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 기능 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FIX] Soniox 전사 실패 로그에 오디오 크기·형식 추가

1 participant