Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
This repository was archived by the owner on Feb 24, 2020. It is now read-only.

image: set the header instead of adding it - #3796

#3796
Merged
iaguis merged 1 commit into
rkt:masterrkt/rkt:masterfrom
JulienBalestra:rkt-fetch-headerJulienBalestra/rkt:rkt-fetch-headerCopy head branch name to clipboard
Sep 18, 2017
Merged

image: set the header instead of adding it#3796
iaguis merged 1 commit into
rkt:masterrkt/rkt:masterfrom
JulienBalestra:rkt-fetch-headerJulienBalestra/rkt:rkt-fetch-headerCopy head branch name to clipboard

Conversation

@JulienBalestra

Copy link
Copy Markdown
Contributor

The go http/client changes its behavior for redirect and header's copy since the go 1.8:

The Client now copies most request headers on redirect.

The ETag header is added and sent twice in the http header request as mentioned in #3678.
This behavior was tracked by ARCH users because they bumped to go1.8 earlier.

As the last release (github v1.28.1) bumped to go1.8, the problem is now present globally.

@lucab lucab left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@lucab

lucab commented Sep 18, 2017

Copy link
Copy Markdown
Member

Thanks for tracking this down to the relevant golang 1.8 behavioral change!

@iaguis
iaguis merged commit 65d9e1c into rkt:master Sep 18, 2017
@lucab lucab added this to the 1.29.0 milestone Sep 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.