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

curl: build with libpsl #11046

Merged
merged 4 commits into from
Jan 11, 2024
Merged

curl: build with libpsl #11046

merged 4 commits into from
Jan 11, 2024

Conversation

imjasonh
Copy link
Member

https://everything.curl.dev/build/deps#libpsl

libpsl

https://rockdaboot.github.io/libpsl/
When you build curl with support for libpsl, the cookie parser knows about the Public Suffix List and thus handle such cookies appropriately.

Signed-off-by: Jason Hall <[email protected]>
@imjasonh imjasonh requested review from kaniini and mattmoor January 11, 2024 21:32
mattmoor
mattmoor previously approved these changes Jan 11, 2024
@imjasonh imjasonh marked this pull request as draft January 11, 2024 21:40
@imjasonh
Copy link
Member Author

Just adding the flag isn't enough, it still builds without libpsl because it's not found:

⚠️  aarch64   | configure: WARNING: libpsl was not found
ℹ️  aarch64   |   PSL:              no      (libpsl not found)

Signed-off-by: Jason Hall <[email protected]>
@imjasonh imjasonh marked this pull request as ready for review January 11, 2024 21:51
@imjasonh
Copy link
Member Author

Added libpsl based on https://git.alpinelinux.org/aports/plain/main/libpsl/APKBUILD, added it as a dep for curl.

@imjasonh imjasonh enabled auto-merge January 11, 2024 21:52
@imjasonh imjasonh added this pull request to the merge queue Jan 11, 2024
@imjasonh
Copy link
Member Author

imjasonh commented Jan 11, 2024

https://github.com/wolfi-dev/os/actions/runs/7494948201/job/20404009550?pr=11046#step:6:2858

ℹ️  aarch64   |   PSL:              enabled

🎉

Merged via the queue into wolfi-dev:main with commit e974bcb Jan 11, 2024
8 checks passed
@imjasonh imjasonh deleted the curl-libpsl branch January 11, 2024 21:58
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.

3 participants