New feature web parser with pattern #950
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Implement WebStreamExtractor to support dynamic URL extraction from web sources in playlist loading. This includes: - New WebStreamExtractor utility class for extracting stream URLs - Updated PlaylistLoader to support '@' prefixed URLs with optional web pattern - Added CMakeLists.txt to include new source files - Added Windows build script for easier compilation
…port Add support for #WEBREGEX: marker to specify custom web stream URL extraction patterns. Improve debug logging to include the web pattern used during extraction.
Modify WebStreamExtractor and PlaylistLoader to enhance URL extraction: - Update regex matching to return the last matched group - Add debug logging for default URL finding process - Adjust marker parsing logic for more flexible group and web regex handling
|
Could you provide some examples? Preferably real world use cases. The purpose of the feature is not clear to me from the PR description. |
|
It is used to search for media URLs in HTML pages using a regex pattern. Example Regex: The current issue is that I can't send requests using custom headers. Which library should I use that works on all platforms? I'm considering using CURL, but does it work on all platforms? |
Add support for #WEBHEADERS: marker to specify custom HTTP headers during web stream URL extraction. Modify WebStreamExtractor and PlaylistLoader to handle and apply custom headers when fetching stream URLs.
Example Usage |
Enhance WebStreamExtractor to handle relative URLs by constructing full URLs when a relative path is detected. This ensures proper URL resolution for web stream sources with partial paths.
- Replace separate #WEBREGEX: and #WEBHEADERS: markers with a unified #WEBPROP: marker - Update PlaylistLoader to store web-related properties directly on the channel - Modify StreamUtils to extract web stream URLs using channel properties - Change header parsing delimiter from '=' to ':' for more flexible header specification - Add 'isWebUrl' and 'isMediaEntry' properties to improve stream URL extraction logic
- Add WebStreamExtractor overload for MediaEntry to support stream URL extraction - Modify URL extraction to preserve original URL parameters - Update logging to include more context during URL extraction - Improve error handling by returning empty string for invalid URLs - Remove redundant 'isMediaEntry' property parsing
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
Implement WebStreamExtractor to support dynamic URL extraction from web sources in playlist loading. This includes:
Missing Features
Example Usage
URL must be specified in parentheses.