Bug 258081 – www.cross-code.com using incompatible ogg format

• Status: RESOLVED
• Resolution: CONFIGURATION CHANGED
• Component: Media

Created attachment 466693 [details]
Error screen before crash

http://www.cross-code.com/en/start

Crashes with errors on any iOS or ipadOS and only in safari in macOS. Works well with chrome on macOS.

[Karl Dubost]

1. Go to http://www.cross-code.com/en/start with STP 172

There is a message saying:

> Untested Browser
> We don't know if your current browser supports CrossCode, so proceed at own risk. If things don't work out, you can try Chrome or Firefox.


According to Firefox console.log
All JSON are sending 

XML Parsing Error: not well-formed
Location: http://www.cross-code.com/data/effects/trail.json
Line Number 1, Column 1:


There is no content type sent. 

```
HTTP/1.1 200 OK
Server: nginx
Date: Thu, 15 Jun 2023 09:41:23 GMT
Content-Length: 9428
Connection: keep-alive
Access-Control-Allow-Origin: *
Last-Modified: Sat, 20 Aug 2016 23:12:56 GMT
ETag: "24d4-53a88f48d135f"
Accept-Ranges: bytes
Cache-Control: max-age=7200, public, must-revalidate
```

Back to Safari to understand.


```
HTTP/1.1 200 OK
ETag: "123c-53a88f1ec395f"
Access-Control-Allow-Origin: *
Last-Modified: Sat, 20 Aug 2016 23:12:12 GMT
Date: Thu, 15 Jun 2023 09:40:40 GMT
Accept-Ranges: bytes
Content-Length: 4668
Cache-Control: max-age=7200, public, must-revalidate
Connection: keep-alive
Server: nginx
```


The error sent by the app is 


```
map: [GAME INIT], version: [---], platform: [Browser], OS: [Mac], browser: [Safari], browserVersion: [17], webAudioSupport: [true], webAudioOption: [null]
Message: [Web Audio Load Error: Could not DECODE: media/sound/environment/train-cube-appear.ogg]

--- GAME INFO ---


STACK TRACE

@http://www.cross-code.com/js/game.compiled.js:95:495
```

so that looks like incompatible ogg format.
http://www.cross-code.com/media/sound/environment/train-cube-appear.ogg

[Alexey Proskuryakov]

If I understand what Karl is saying correctly, this seems like an issue in system frameworks below WebKit. 

Could you please report it to Apple via https://feedbackassistant.apple.com, and post the feedback number here? We can also migrate it ourselves, but you'll have better visibility into its state if you report the feedback yourself.

[Radar WebKit Bug Importer]

<rdar://problem/111111557>

[Karl Dubost]

Testing http://www.cross-code.com/media/sound/environment/train-cube-appear.ogg

Firefox plays the file.
http://www.cross-code.com/media/sound/environment/train-cube-appear.ogg
Safari, Chrome downloads it.

This is due to the mime type.

Safari and Chrome do not support application/x-ogg
https://source.chromium.org/search?q=%22%2Fx-ogg%22&ss=chromium%2Fchromium%2Fsrc:chrome%2F

But they both support 
audio/ogg
https://source.chromium.org/search?q=%22%2Fogg%22&sq=&ss=chromium%2Fchromium%2Fsrc:chrome%2F


Firefox support both

https://searchfox.org/mozilla-central/rev/05b595d95ddb78d660860e8edc0f9b26d38508fc/devtools/shared/network-observer/NetworkHelper.sys.mjs#458-460
```js
"audio/ogg": "media",
"application/ogg": "media",
"application/x-ogg": "media",
```


That said on all platforms I can play the game. 
Not sure how this is handled. There is no crash anymore.

The game has also a reference to
http://www.cross-code.com/js/game.compiled.js

```
            FYa: {
                C1: "ogg",
                M3: "audio/ogg; codecs=vorbis"
            },
```

[Karl Dubost]

furrows-lumber.0c do you still have the issue on Safari?

[Karl Dubost]

The unsupported error message is due to 
https://www.cross-code.com/page/js/browser.js

```
    window.doBrowserCheck = function(target, onConfirm){
        BrowserDetect.init();
        var browserName = BrowserDetect.browser;
        var notification = null;

        if(BrowserDetect.OS == "iOS" || BrowserDetect.OS == "Android"){
            notification = "mobile";
        }
        else{
            switch(BrowserDetect.browser){
                case "Chrome":
                    break;
                case "Firefox":
                    if(BrowserDetect.version < 31){
                        notification = "oldversion";
                    }
                    break;
                case "Explorer":
                    notification = "notyetsupported";
                    break;
                case "Opera":
                    if(BrowserDetect.version < 15)
                        notification = "oldversion";
                    break;
                case "Safari":
                    notification = "untested";
                    break;
                default:
                    notification = "untested";
                    break;
            }
        }
```

And seems to be totally unnecessary. So it would be great to reach out the developers of the game.