Project

Profile

Help

Issue #4646

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

Added by dkliban@redhat.com over 1 year ago. Updated 6 months ago.

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

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

Associated revisions

Revision b8cca325 View on GitHub
Added by ipanova@redhat.com about 1 year ago

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

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

Revision b8cca325 View on GitHub
Added by ipanova@redhat.com about 1 year ago

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

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

Revision b8cca325 View on GitHub
Added by ipanova@redhat.com about 1 year ago

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

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

Revision b8cca325 View on GitHub
Added by ipanova@redhat.com about 1 year ago

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

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

History

#1 Updated by amacdona@redhat.com over 1 year ago

  • Description updated (diff)

#2 Updated by ipanova@redhat.com over 1 year ago

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

#3 Updated by ipanova@redhat.com over 1 year ago

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

#4 Updated by dkliban@redhat.com about 1 year ago

  • Triaged changed from No to Yes

#5 Updated by bmbouter about 1 year ago

  • Tags deleted (Pulp 3)

#6 Updated by ipanova@redhat.com about 1 year ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ipanova@redhat.com

#7 Updated by ipanova@redhat.com about 1 year ago

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

#8 Updated by ipanova@redhat.com about 1 year ago

  • Status changed from POST to MODIFIED

Applied in changeset commit:pulp_docker|b8cca32518f9057b441e599200bfa5a6dfa18eda.

#10 Updated by ipanova@redhat.com 8 months ago

  • Project changed from Docker Support to Container Support

#11 Updated by ipanova@redhat.com 7 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#12 Updated by ipanova@redhat.com 6 months ago

  • Sprint/Milestone set to 1.0.0

Please register to edit this issue

Also available in: Atom PDF