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

Nginx passenger updates #68

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

zealot128
Copy link
Contributor

See new readme :)

The dump from our adjustments over the year.
Most notably:

  • ActionCable/MessageBus
  • various variables can now be set
  • Cache folder can be set

* can now programatically add nginx conf.d files to /etc/nginx/conf.d/*
  * gzip.conf - gzip related config
  * passenger.conf - all passenger variables
  * privacy.conf - disable nginx Powered by header
* add recommended "message-bus" hosting with passenger, as a seperate
  process. NOTE: add some snippet to application.rb to make it work:
  https://github.com/SamSaffron/message_bus#passenger
* Added various variables
* new var: nginx_max_body_size
* new var: nginx_public_folder_to_cache
* new var: nginx_site_config for total customization of site-conf
*
@arBmind
Copy link
Member

arBmind commented Jun 3, 2018

This seems to be a nice update. I only found one issue:

The added nginx/passenger/templates/nginx.conf.j2 seems to duplicate the nginx/server/templates/nginx.conf.j2 while it is less configurable.

Are you not using our nginx/server sub-module?

@zealot128
Copy link
Contributor Author

zealot128 commented Apr 29, 2019

No, i've configuring the nginx completely with the role (maybe still in my fork).

The DW-nginx role add's its own nginx ppa, which collides with the nginx passenger's.

$ sudo apt-get install -y libnginx-mod-http-passenger
The following packages have unmet dependencies:
 libnginx-mod-http-passenger : Depends: nginx-common (< 1.14.1) but 1.14.2-0+bionic1 is to be installed
E: Unable to correct problems, you have held broken packages.

I've might need to add a boolean flag to the nginx/server-role that disables the nginx own ppa for that case.

@zealot128 zealot128 mentioned this pull request Apr 29, 2019
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

Successfully merging this pull request may close these issues.

2 participants