-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[docker-19.03] docker 19.03 / v0.6.2 cherry picks #1178
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Tibor Vass <tibor@docker.com> (cherry picked from commit 77ca4bc)
Signed-off-by: Tibor Vass <tibor@docker.com> (cherry picked from commit bd8f49e)
Signed-off-by: Tibor Vass <tibor@docker.com> (cherry picked from commit aa3ad5f)
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> (cherry picked from commit bc3a1ee)
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> (cherry picked from commit 1d7e7a6)
If a single ETag is requested in `If-None-Match`, some servers do not include that (unambiguous) ETag header in the response. For detailed description, see: moby#905 (comment) Signed-off-by: Robert Estelle <robertestelle@gmail.com> (cherry picked from commit c63c6f9)
Otherwise a 200 response without an ETag could be incorrectly associated to previous content in the following scenario: * The remote server had in the past responded with an ETag for this resource, which was cached. - (Otherwise, onlyETag would be empty) * That was the only ETag cached for this resource. - (Otherwise, onlyETag would be empty) * The remote server then stopped supporting ETag/If-None-Match for this resource at all. - (Otherwise, it would respond with a 304 or a 200+ETag) Signed-off-by: Robert Estelle <robertestelle@gmail.com> (cherry picked from commit 6ff0868)
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> (cherry picked from commit eede0fa)
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> (cherry picked from commit e3e13ad)
c35f47a
to
5b736e3
Compare
Signed-off-by: Tonis Tiigi <tonistiigi@gmail.com> (cherry picked from commit 0ef4356)
5b736e3
to
2a9b6b5
Compare
tiborvass
approved these changes
Sep 21, 2019
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix socket handling #1144
session: release forwarded ssh socket connection per connection #1150
llbsolver: fix error on multiple cache importers #1156
http: Handle missing but unambiguous ETags in response #1159
solver: fix pipe signaling on incoming updates #1166
vendor: update go-runc to e029b79d #1168