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

[v2.2.0] output.harbor OGG container broken #3265

Closed
DigiBC opened this issue Jul 26, 2023 · 0 comments · Fixed by #3276
Closed

[v2.2.0] output.harbor OGG container broken #3265

DigiBC opened this issue Jul 26, 2023 · 0 comments · Fixed by #3276

Comments

@DigiBC
Copy link

DigiBC commented Jul 26, 2023

Describe the bug

As already confirmed in #3255 there's another issue with output.harbor in version 2.2.0:
Using the OGG container results in an empty or garbled stream, even though the log doesn't indicate any problems after successful connection with a streaming client.
Despite connection, none of the tested streaming clients could render the stream.
(By the way, other streams like MP3 work flawlessly.)

To Reproduce

I've used the following simple code examples:

output.harbor(%vorbis, sine(), mount="/stream")
output.harbor(%ogg(%flac), sine(), mount="/stream")

Streaming clients like VLC report errors:

ogg debug: File length is 0 bytes
ogg warning: couldn't find any ogg logical stream

CURL shows the following:

~$ curl -v --http0.9 http://localhost:8000/stream
*   Trying 127.0.0.1:8000...
* Connected to localhost (127.0.0.1) port 8000 (#0)
> GET /stream HTTP/1.1
> Host: localhost:8000
> User-Agent: curl/7.81.0
> Accept: */*
> 
* Mark bundle as not supporting multiuse
< HTTP/1.1 200 OK
< Content-type: application/ogg
* no chunk, no close, no size. Assume close to signal end
< 

Expected behavior

It worked flawlessly in version 2.1.4 and earlier.

Version details

  • OS: Linux Mint 21.2 (Ubuntu 22.04 'Jammy') amd64
  • Version: Liquidsoap Rolling Release 2.2.1 (e3b25cd)

Install method

Install of intermediate package 'liquidsoap-e3b25cd_2.2.1-ubuntu-jammy-1_amd64'

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 a pull request may close this issue.

1 participant