Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Browsers don't sniff basic audio #151

Closed
GPHemsley opened this issue Aug 29, 2021 · 2 comments · Fixed by #153
Closed

Browsers don't sniff basic audio #151

GPHemsley opened this issue Aug 29, 2021 · 2 comments · Fixed by #153

Comments

@GPHemsley
Copy link
Member

Neither Firefox nor Chromium sniff for the string ".snd", the basic audio signature (audio/basic). I'm not even sure that Chromium knows that such a file format exists. (Firefox at least defines AUDIO_BASIC as a MIME type constant.)

Given this, I'm guessing we can remove the signature from the spec.

GPHemsley added a commit to GPHemsley/mimesniff that referenced this issue Sep 3, 2021
@GPHemsley
Copy link
Member Author

From RFC 2046:

4.3. Audio Media Type

A media type of "audio" indicates that the body contains audio data.
Although there is not yet a consensus on an "ideal" audio format for
use with computers, there is a pressing need for a format capable of
providing interoperable behavior.

The initial subtype of "basic" is specified to meet this requirement
by providing an absolutely minimal lowest common denominator audio
format. It is expected that richer formats for higher quality and/or
lower bandwidth audio will be defined by a later document.

The content of the "audio/basic" subtype is single channel audio
encoded using 8bit ISDN mu-law [PCM] at a sample rate of 8000 Hz.

Unrecognized subtypes of "audio" should at a miniumum be treated as
"application/octet-stream". Implementations may optionally elect to
pass subtypes of "audio" that they do not specifically recognize to a
robust general-purpose audio playing application, if such an
application is available.

This description is repeated in the audio/basic registration.

I'm not sure this was a format that was ever even seriously used.

GPHemsley added a commit that referenced this issue Sep 9, 2021
@GPHemsley
Copy link
Member Author

gopherbot pushed a commit to golang/go that referenced this issue Apr 20, 2022
The WHATWG Mime Sniffing Standard we follow dropped support for
.snd / audio/basic.
whatwg/mimesniff#151

Change-Id: Iae1bd8b29063b06b5b3909a944e12ead5974a526
Reviewed-on: https://go-review.googlesource.com/c/go/+/400754
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Auto-Submit: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
dteh pushed a commit to dteh/fhttp that referenced this issue Jun 10, 2022
The WHATWG Mime Sniffing Standard we follow dropped support for
.snd / audio/basic.
whatwg/mimesniff#151

Change-Id: Iae1bd8b29063b06b5b3909a944e12ead5974a526
Reviewed-on: https://go-review.googlesource.com/c/go/+/400754
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
Run-TryBot: Brad Fitzpatrick <bradfitz@golang.org>
Auto-Submit: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Ian Lance Taylor <iant@google.com>
Auto-Submit: Ian Lance Taylor <iant@google.com>
Reviewed-by: Damien Neil <dneil@google.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants