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: show ipfs and ipns as supported "protocols" #12508

Closed
wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Dec 12, 2023

They are accepted schemes in URLs passed to curl.

Reported-by: Chara White
Bug: https://curl.se/mail/archive-2023-12/0026.html

@jay
Copy link
Member

jay commented Dec 12, 2023

i changed the logic to account for insertion if there is no imap support or no other protocols follow http.

also test 1013 will need to be fixed

2023-12-12T17:09:35.7833979Z test 1013...[Compare curl --version with curl-config --protocols]
2023-12-12T17:09:36.6960567Z Mismatch in protocols lists:
2023-12-12T17:09:36.6961577Z curl:        dict file ftp ftps gopher gophers http https imap imaps ipfs ipns ldap ldaps mqtt pop3 pop3s rtsp smtp smtps telnet tftp
2023-12-12T17:09:36.6963534Z curl-config: dict file ftp ftps gopher gophers http https imap imaps ldap ldaps mqtt pop3 pop3s rtsp smtp smtps telnet tftp

They are accepted schemes in URLs passed to curl (the tool, not the
library).

Also makes curl-config show the same list.

Co-Authored-by: Jay Satiro
Reported-by: Chara White
Bug: https://curl.se/mail/archive-2023-12/0026.html
Closes #12508
@bagder bagder closed this in 102de7a Dec 15, 2023
@bagder bagder deleted the bagder/curl-ipfs branch December 15, 2023 13:04
vszakats added a commit to curl/curl-for-win that referenced this pull request Dec 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

2 participants