Skip to content
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

Remove dependency on docker_content_digest digest headers for docker updates #8010

Open
yeikel opened this issue Sep 11, 2023 · 0 comments
Open

Comments

@yeikel
Copy link
Contributor

yeikel commented Sep 11, 2023

Code improvement description

Depending on this header to get around rate limits is not desired because this header is not present in every Docker Registry Implementation:

Because of the origins this specification, the client MAY encounter Docker-specific headers, such as Docker-Content-Digest, or Docker-Distribution-API-Version. These headers are OPTIONAL and clients SHOULD NOT depend on them.

In my opinion, what we should do instead is to retry

Additional Context :

deitch/docker_registry2#98

#7578 (comment)

@yeikel yeikel changed the title Remove dependency on docker_content_digest digest headers Remove dependency on docker_content_digest digest headers for docker updates Sep 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant