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

New Feature: Web parser with pattern for Piers #957

Open
wants to merge 3 commits into
base: Piers
Choose a base branch
from

Conversation

alpgul
Copy link

@alpgul alpgul commented Feb 8, 2025

Adapts the "Web Parser with Regex #950 " feature, originally developed for the Omega version, to the Piers version.

- Update zlib source to version 1.3.1
- Modify CMake configuration to support static library build
- Disable example binaries
- Update SHA256 checksum
@alpgul alpgul changed the title Piers web parser with pattern New Feature: Web parser with pattern for Piers Feb 8, 2025
@phunkyfish
Copy link
Member

Thanks for all the updates, can you merge the last 3 commits and I'll do a full review in the coming days?

@phunkyfish
Copy link
Member

Once this passes review the last step will be to add a changelog entry and mini version increment (the middle number as its a new feature) in addon.xml.in so it can be merged and released.

Improve web stream URL extraction handling in StreamUtils

Remove Base64 and CurlUtils utility classes

Clean up CurlUtils by removing unused methods and Base64 dependency, and update CMakeLists.txt to remove Base64 source and header files
@alpgul alpgul force-pushed the Piers-webParserWithPattern branch from f6d9812 to 22486fe Compare February 14, 2025 14:40
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 this pull request may close these issues.

2 participants