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

alt-svc: enable by default #5868

Closed
wants to merge 1 commit into from
Closed

alt-svc: enable by default #5868

wants to merge 1 commit into from

Conversation

bagder
Copy link
Member

@bagder bagder commented Aug 26, 2020

No longer considered experimental

Checklist:

  • remove CURLALTSVC_IMMEDIATELY as it isn't implemented
  • works with curl_easy_duphandle() - #5923 landed

Proposal taken to the mailing list: https://curl.haxx.se/mail/lib-2020-08/0033.html

@bagder bagder added the HTTP label Aug 26, 2020
bagder added a commit that referenced this issue Aug 28, 2020
No longer considered experimental

Closes #5868
@HoneyryderChuck
Copy link

@HoneyryderChuck HoneyryderChuck commented Aug 28, 2020

does the alt-svc feature also support the HTTP/2 alt-svc frame? Does any server actually use it?

@bagder
Copy link
Member Author

@bagder bagder commented Aug 28, 2020

does the alt-svc feature also support the HTTP/2 alt-svc frame?

No, unfortunately it doesn't.

Does any server actually use it?

I don't know...

bagder added a commit that referenced this issue Sep 5, 2020
[Remove CURLALTSVC_IMMEDIATELY]

alt-svc support is no longer considered experimental

Closes #5868
@bagder bagder force-pushed the bagder/enable-altsvc branch from 8c1278f to dd2145e Compare Sep 5, 2020
bagder added a commit that referenced this issue Sep 6, 2020
[Remove CURLALTSVC_IMMEDIATELY]

alt-svc support is no longer considered experimental

Closes #5868
@bagder bagder force-pushed the bagder/enable-altsvc branch from dd2145e to 5dd2628 Compare Sep 6, 2020
bagder added a commit that referenced this issue Sep 19, 2020
[Remove CURLALTSVC_IMMEDIATELY]

alt-svc support is no longer considered experimental

Closes #5868
@bagder bagder marked this pull request as ready for review Oct 21, 2020
bagder added a commit that referenced this issue Oct 21, 2020
[Remove CURLALTSVC_IMMEDIATELY]

alt-svc support is no longer considered experimental

Closes #5868
bagder added a commit that referenced this issue Oct 22, 2020
Remove CURLALTSVC_IMMEDIATELY, which was never implemented/supported.

alt-svc support in curl is no longer considered experimental

Closes #5868
@xquery
Copy link
Member

@xquery xquery commented Oct 22, 2020

looks good to me

Remove CURLALTSVC_IMMEDIATELY, which was never implemented/supported.

alt-svc support in curl is no longer considered experimental

Closes #5868
@bagder
Copy link
Member Author

@bagder bagder commented Oct 24, 2020

The CI jobs are in a very sorry state right now but independent of this work. Since there's no short term fix for that, I'm going to merge this PR anyway really soon.

@bagder bagder deleted the bagder/enable-altsvc branch Oct 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants