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

unknown directive "more_set_headers" in /etc/nginx/conf.d/registry.conf:30 #452

Closed
polym opened this issue Apr 28, 2015 · 9 comments
Closed

Comments

@polym
Copy link

polym commented Apr 28, 2015

I used contrib/compose/ to build up my private registry, but something wrong occured in compose_nginx container. I search for this problem, and find HttpHeadersMoreModule. So maybe just FROM: nginx:1.7 is not enough. We should add module to nginx and compile it.

@stevvooe
Copy link
Collaborator

@polym You can use add_header with the "always" parameter after 1.7.5 in nginx. We are a little annoyed with nginx, as well, since this is an odd restriction. I've file #460 to cover this.

Let me know how you'd like to proceed.

@polym
Copy link
Author

polym commented Apr 29, 2015

@stevvooe Thanks. I'm new to docker-registry, and try it by using document. :)

@polym polym closed this as completed Apr 29, 2015
stevvooe referenced this issue May 2, 2015
Signed-off-by: Mary Anthony <mary@docker.com>
@macsteps
Copy link

macsteps commented May 8, 2015

Comment out or remove more_set_headers line, and add

add_header 'Docker-Distribution-Api-Version:' 'registry/2.0' always;

@kentoj
Copy link

kentoj commented May 13, 2015

I hit this same error today. Is there a fork with a PR to fix this yet?

@stevvooe
Copy link
Collaborator

@kentoj How do you propose we fix a bug in nginx with a fork of distribution?

@kentoj
Copy link

kentoj commented May 13, 2015

@stevvooe it looks like I must have misunderstood the issue. If @macsteps suggestion is not a solution to this problem then I don't know how to fix this.

@stevvooe
Copy link
Collaborator

@kentoj We need to get the documentation in order on this. @macsteps solution is the right approach but requires nginx 1.7.5+.

danzel added a commit to Smartrak/docker-registry-build that referenced this issue May 14, 2015
@thaJeztah
Copy link
Member

@stevvooe for people using a containerised NGINX; 1.9 is now the only supported NGINX version on the registry; https://registry.hub.docker.com/_/nginx/

@stevvooe
Copy link
Collaborator

@thaJeztah Looks like we should move the documentation to fully embrace the add_headers directive. This will clean things up a bit. Thank you for the update.

cc @moxiegirl

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

5 participants