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

redirect handling by protocol handler #16075

Closed
wants to merge 2 commits into from

Conversation

icing
Copy link
Contributor

@icing icing commented Jan 23, 2025

Adds a follow() callback to protocol handlers, so they may decide how to act on a newurl after a request has been done. This is optional.

This moves the HTTP code for handling redirects from multi.c to http.c where it should be. If we ever add a protocol with its own logic, it would install its own follow function.

icing added 2 commits January 23, 2025 11:48
Adds a `follow()` callback to protocol handlers, so they
may decide how to act on a `newurl` after a request has
been done. This is optional.

This moves the HTTP code for handling redirects from multi.c
to http.c where it should be. If we ever add a protocol with
its own logic, it would install its own follow function.
@icing icing added the HTTP label Jan 23, 2025
Copy link
Member

@bagder bagder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I like moving HTTP specific functions away from generic code into HTTP specific!

lib/http.c Show resolved Hide resolved
@bagder bagder closed this in 1213c31 Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

2 participants