Skip to content

fix(hub): accept WAV MIME alias in validation - #108

Open
gmolto wants to merge 1 commit into
develfrom
gmolto-kokoro-wav-mime
Open

gmolto wants to merge 1 commit into
develfrom
gmolto-kokoro-wav-mime

Conversation

@gmolto

@gmolto gmolto commented Oct 2, 2026

Copy link
Copy Markdown
Member

Summary

  • Treat audio/wav and audio/wave as equivalent when validating WAV output media types.
  • Add regression coverage confirming WAV signatures detected by Go's http.DetectContentType pass for either declaration and do not match unrelated media types.
  • This issue was identified while running the kokoro-tts acceptance test from grycap/oscar-hub: the service generated the audio successfully, but validation expected audio/wav while detection reported audio/wave.

Verification

  • go test ./pkg/hub
  • Re-ran ocli-dev hub validate kokoro-tts against the local cluster using this change: asynchronous acceptance test passed, including downloaded output detected as audio/wave.

@gmolto gmolto self-assigned this Oct 2, 2026
@gmolto
gmolto requested a review from RK181 October 2, 2026 16:42

@RK181 RK181 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

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.

2 participants