Skip to content

Make it faster - #106

Open
ebb-earl-co wants to merge 9 commits into
developfrom
make-it-faster
Open

Make it faster#106
ebb-earl-co wants to merge 9 commits into
developfrom
make-it-faster

Conversation

@ebb-earl-co

Copy link
Copy Markdown
Owner

This pull request attempts to change the fundamental library of the project, requests, to something faster, addressing #97

* Refactor to use httpx instead of Requests/Niquests

* Fix video.py and hls.py handling of M3U8 manifests

* Fix bug in hls.variant_streams()
@ebb-earl-co ebb-earl-co added enhancement New feature or request python Pull requests that update Python code labels Mar 13, 2024
@ebb-earl-co ebb-earl-co self-assigned this Mar 13, 2024
@ebb-earl-co
ebb-earl-co changed the base branch from trunk to develop March 13, 2024 20:59
Comment thread tidal_wave/video.py Outdated
self.M3U8 = m3u8.load(playlist.uri)
if self.M3U8 is None or len(self.M3U8.files) == 0:
m3u8_files: List[str] = variant_streams(self.m3u8, client, return_urls=True)
if not all(file.startswith("http://vmz-ad-cf.video.tidal.com") for file in m3u8_files):

Check failure

Code scanning / CodeQL

Incomplete URL substring sanitization

The string [http://vmz-ad-cf.video.tidal.com](1) may be at an arbitrary position in the sanitized URL.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request python Pull requests that update Python code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants