Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #4646

closed

'docker-content-digest' header is not returned with requests for a tag

Added by dkliban@redhat.com over 5 years ago. Updated almost 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
ipanova@redhat.com
Sprint/Milestone:
1.0.0
Start date:
Due date:
Estimated time:
Severity:
2. Medium
Platform Release:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Sprint 55
Quarter:

Description

Crane and Pulp don't set the 'Docker-content-digest' header [0]. Users want to make a HEAD request for a tag to find out if that tag is pointing to a new manifest. Without this header, users must download the manifest in order to learn its digest.

The first thing that I found when searching the web is a bug about Crane[1]. We need to fix this for Pulp 3.

[0] https://docs.docker.com/registry/spec/api/#content-digests
[1] https://github.com/renovatebot/renovate/issues/2315

Actions
Copy link
 #1

Updated by amacdona@redhat.com over 5 years ago

  • Description updated (diff)
Actions
Copy link
 #2

Updated by ipanova@redhat.com over 5 years ago

this headers is optional, in addition docs state that you should not rely on this header because it is not trusted

Actions
Copy link
 #3

Updated by ipanova@redhat.com over 5 years ago

however for compatibility of clients it is better to include the header whenever the digest is available.

Actions
Copy link
 #4

Updated by dkliban@redhat.com over 5 years ago

  • Triaged changed from No to Yes
Actions
Copy link
 #5

Updated by bmbouter over 5 years ago

  • Tags deleted (Pulp 3)
Actions
Copy link
 #6

Updated by ipanova@redhat.com over 5 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com
Actions
Copy link
 #7

Updated by ipanova@redhat.com over 5 years ago

  • Status changed from ASSIGNED to POST
  • Sprint set to Sprint 55

Added by ipanova@redhat.com over 5 years ago

Revision b8cca325 | View on GitHub

Add 'Docker-Content-Digest' header to the reasponse headers.

closes #4646 https://pulp.plan.io/issues/4646

Added by ipanova@redhat.com over 5 years ago

Revision b8cca325 | View on GitHub

Add 'Docker-Content-Digest' header to the reasponse headers.

closes #4646 https://pulp.plan.io/issues/4646

Added by ipanova@redhat.com over 5 years ago

Revision b8cca325 | View on GitHub

Add 'Docker-Content-Digest' header to the reasponse headers.

closes #4646 https://pulp.plan.io/issues/4646

Added by ipanova@redhat.com over 5 years ago

Revision b8cca325 | View on GitHub

Add 'Docker-Content-Digest' header to the reasponse headers.

closes #4646 https://pulp.plan.io/issues/4646

Actions
Copy link
 #8

Updated by ipanova@redhat.com over 5 years ago

  • Status changed from POST to MODIFIED

Applied in changeset commit:pulp_docker|b8cca32518f9057b441e599200bfa5a6dfa18eda.

Actions
Copy link
 #10

Updated by ipanova@redhat.com about 5 years ago

  • Project changed from Docker Support to Container Support
Actions
Copy link
 #11

Updated by ipanova@redhat.com about 5 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions
Copy link
 #12

Updated by ipanova@redhat.com almost 5 years ago

  • Sprint/Milestone set to 1.0.0
Actions
Send by e-mail Copy link

Also available in: Atom PDF