Project

Profile

Help

Issue #4646

closed

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

Added by dkliban@redhat.com over 3 years ago. Updated over 2 years 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
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 #1

Updated by amacdona@redhat.com over 3 years ago

  • Description updated (diff)
Actions #2

Updated by ipanova@redhat.com over 3 years ago

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

Actions #3

Updated by ipanova@redhat.com over 3 years ago

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

Actions #4

Updated by dkliban@redhat.com over 3 years ago

  • Triaged changed from No to Yes
Actions #5

Updated by bmbouter over 3 years ago

  • Tags deleted (Pulp 3)
Actions #6

Updated by ipanova@redhat.com over 3 years ago

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

Updated by ipanova@redhat.com over 3 years ago

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

Added by ipanova@redhat.com over 3 years ago

Revision b8cca325

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

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

Added by ipanova@redhat.com over 3 years ago

Revision b8cca325

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

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

Added by ipanova@redhat.com over 3 years ago

Revision b8cca325

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

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

Added by ipanova@redhat.com over 3 years ago

Revision b8cca325

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

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

Actions #8

Updated by ipanova@redhat.com over 3 years ago

  • Status changed from POST to MODIFIED

Applied in changeset commit:pulp_docker|b8cca32518f9057b441e599200bfa5a6dfa18eda.

Actions #10

Updated by ipanova@redhat.com almost 3 years ago

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

Updated by ipanova@redhat.com almost 3 years ago

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

Updated by ipanova@redhat.com over 2 years ago

  • Sprint/Milestone set to 1.0.0

Also available in: Atom PDF