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

Jump to bottom

http: add custom empty headers to repeated requests #1920

Closed
wants to merge 1 commit into from
Closed

http: add custom empty headers to repeated requests #1920

wants to merge 1 commit into from

Conversation

mkauf
Copy link
Contributor

@mkauf mkauf commented Sep 24, 2017

No description provided.

@bagder
Copy link
Member

bagder commented Sep 24, 2017

Maybe add a test for this?

@mkauf
Copy link
Contributor Author

mkauf commented Sep 25, 2017

The PR already includes an improved test4, with two requests instead of one.

@bagder
Copy link
Member

bagder commented Sep 25, 2017

Oops, sorry for being sloppy. And thanks for being on the ball! 👍

@mkauf mkauf closed this in eac324f Sep 28, 2017
@mkauf
Copy link
Contributor Author

mkauf commented Sep 28, 2017

Thanks for reviewing 👍

@mkauf mkauf deleted the custom_header_bugfix branch September 28, 2017 19:29
@mkauf mkauf added the HTTP label Sep 28, 2017
@lock lock bot locked as resolved and limited conversation to collaborators Jan 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@bagder bagder bagder approved these changes

Assignees
No one assigned
Labels
HTTP
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@mkauf @bagder