Groups include filter - Githubissues

sonroyaalmerol / m3u-stream-merger-proxy

A lightweight HTTP proxy server dockerized for consolidating and streaming content from multiple IPTV M3U playlists, acting as a load balancer between provided sources.

https://hub.docker.com/r/sonroyaalmerol/m3u-stream-merger-proxy

12 stars 1 forks source link

Groups include filter #41

Closed andjos closed 3 months ago

andjos commented 3 months ago

Context

Include only selected channel groups to parse. Often lots of unwanted groups in playlist, taking time to parse.

Choices

Just checking during parsing if channel groups should be included or not, default is all.

Test instructions

Test run on several huge playlists.

Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I've added documentation about this change to the README.
[x] I've not introduced breaking changes.

sonroyaalmerol commented 3 months ago

Looks good! Do you mind fixing the function name typo? Should be checkIncludeGroup.

Another concern is the M3U "spec" for IPTV playlists, if that even exists. Is the group-title value usually case-insensitive? I would prefer leaving the actual value checking as case-sensitive if there's no problem with it.

andjos commented 3 months ago

Agree we should follow the "spec", will fix that and the typo.

sonroyaalmerol commented 3 months ago

Looks good! Thanks!