Skip to content

Add IN_BAND stream type to audio format enums#175

Open
OzGav wants to merge 4 commits intomainfrom
OzGav-patch-1
Open

Add IN_BAND stream type to audio format enums#175
OzGav wants to merge 4 commits intomainfrom
OzGav-patch-1

Conversation

@OzGav
Copy link
Contributor

@OzGav OzGav commented Mar 4, 2026

Add IN_BAND stream type for audio formats that carry metadata inside the container (e.g. Ogg Vorbis comments, FLAC metadata blocks, Opus tags). For example, there are Icecast servers streaming in OGG format with metadata in standard Ogg tags that update with each new track.

Add OGG stream type to enums for audio formats.. The reason is a user is streaming from an icecast server. It is being sent in OGG format with metadata in standard Ogg tags that get updated with each new track. Assuming we wnt to support this then the first step is to add this new streamtype and once that is in the new code path to support this will be added to core
@OzGav OzGav added the enhancement Enhancement of an existing feature/functionality label Mar 4, 2026
Replaced OGG stream type with IN_BAND stream type in enums.
@OzGav OzGav changed the title Add OGG stream type to audio format enums Add IN_BAND stream type to audio format enums Mar 5, 2026
OzGav added 2 commits March 5, 2026 12:58
Updated the comment for IN_BAND to clarify its purpose.

# in_band: radio stream with in-band metadata
# (e.g. ogg vorbis comments, flac metadata blocks, opus tags)
IN_BAND = "in_band"
Copy link
Contributor

Choose a reason for hiding this comment

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

This feels a bit weird as its not a stream type but more a behavior of the stream.
How are you seeing this ? Is this going to be processed differently ?

Copy link
Contributor Author

@OzGav OzGav Mar 11, 2026

Choose a reason for hiding this comment

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

I was thinking of doing the same as we did for shoutcast so once this type is identified this is set and it allows a dedicated code path for handling? So this is about supporting metadata in radio streams. Originally I was going to call it just OGG but then discovered FLAC and Opus support this as well.

Copy link
Contributor

Choose a reason for hiding this comment

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

So, we are talking about stream type here (not content type) so OGG or any other codec would be wrong

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah fair enough. Yes we are talking about a stream type that has metadata embedded in tags within it (distinct from ICY or SHOUTCAST). It could be FLAC, OGG, or OPUS codec

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Here is the PR for the actual application of this music-assistant/server#3326

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Enhancement of an existing feature/functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants