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

with the default attrbiutes set nexus/nginx will not work #79

Open
ramonskie opened this issue May 7, 2014 · 2 comments
Open

with the default attrbiutes set nexus/nginx will not work #79

ramonskie opened this issue May 7, 2014 · 2 comments

Comments

@ramonskie
Copy link
Contributor

this nexus cookbook will not work out of the box due to the following settings

default[:nexus][:context_path]                                 = '/nexus'
default[:nexus][:app_server_proxy][:ssl][:port]                = 8443

context_path will not work because it does not correspond with

default[:nexus][:app_server_proxy][:port] = "http://127.0.0.1:#{node[:nexus][:port]}"

this should be

default[:nexus][:app_server_proxy][:port] = "http://127.0.0.1:#{node[:nexus][:port]}#{node[:nexus][:context_path]}"

and the its still a mystery to me why the ssl port is set to 8443

if the http to https redirect will even work
this line https://github.com/RiotGames/nexus-cookbook/blob/master/templates/default/nexus_proxy.nginx.conf.erb#L7

needs to be changed to

return 301 https://$host:8443$request_uri; 
@KAllan357
Copy link
Contributor

I'm not sure what's not working out of the box here. I have been consistently using this cookbook for months, and after issuing a vagrant up, I have been able to hit https://localhost:8443/nexus/index.html without issue.

Of course, if there is an issue, it is definitely worth fixing. As for 8443, when this cookbook was originally written, I might have had some trouble with getting nginx on 443 (despite root). 8443 is probably worth revisiting, but hasn't been a high priority for us.

The redirect came from within the community, and could be incorrect. Also worth fixing IMO.

@ramonskie
Copy link
Contributor Author

okay there is the confusion i thought the idea was when you hit http://localhost/
you would be redirected to nexus

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