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

v2/client: trim content-type parameters #83

Open
lucab opened this issue Jan 14, 2019 · 3 comments
Open

v2/client: trim content-type parameters #83

lucab opened this issue Jan 14, 2019 · 3 comments

Comments

@lucab
Copy link
Member

lucab commented Jan 14, 2019

This is a followup to #82 (comment).

Current docker-registry specification and implementation have some quirks related to content-type headers, returning charset parameters even for media-types that don't allow any.

As a minor compatibility fix, we should revert #82 and instead introduce a mime_trim_params that expunges all parameters from a media type.

@schrieveslaach
Copy link
Contributor

@lucab, I noticed that there are also errors with Nexus 3 (see aixigo/PREvant#44). Is this issue related to it?

@lucab
Copy link
Member Author

lucab commented Jun 3, 2020

@schrieveslaach no, your troubles seem unrelated to this specific issue.
I think you are just hitting some content-negotiation bug, and it wouldn't be the first registry we encounter with some buggy logic.
It would be helpful to have a minimal Rust example to look at and a trace at the HTTP level.

@schrieveslaach
Copy link
Contributor

@lucab, sorry for interfering. It is an issue of Nexus 3.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants