AnoopAlias / AUTOM8N

AUTOM8N is a WHM/cPanel plugin providing a DNS load-balanced, high available, horizontally scaling webstack and offering active-active-redundancy-high-availability-and-native-nginx-on-cpanel
https://autom8n.com/
GNU General Public License v3.0
107 stars 53 forks source link

Add Template for Silverstripe v4 #629

Closed purplespider closed 4 years ago

purplespider commented 4 years ago

The current "SilverStripe" template seems to only work for older SilverStripe version 3 sites.

Would it be possible to add a template for the newer SilverStripe v4?

The main difference is that the public root is no longer public_html but is now public_html/public.

There appears to be an official recipe for SilverStripe 4 here: https://www.nginx.com/resources/wiki/start/topics/recipes/silverstripe/

Thanks!

AnoopAlias commented 4 years ago

The template is added and will be available in the next RPM upgrade

AnoopAlias commented 4 years ago

Just FYI .. the new RPM is available in the repo with this template added

purplespider commented 4 years ago

Thanks for adding this Anoop, but I'm struggling to get it to work. It may just be that the public root of a Silverstripe 4 site should be: {{ DOCUMENTROOT }}/public rather than just {{ DOCUMENTROOT }} (presume DOCUMENTROOT is public_html?)

I'm happy to refine and test this for you, by using a custom nginx.conf in the site root, but I keep getting Invalid/Require Reload, even when using the basic template from here: https://autom8n.com/docs/Application_template.html

So if you can help with a valid starting point, I'll do the rest!

xlightwaverx commented 4 years ago

Hi James,

You can make this change in /opt/nDeploy/conf/5050.j2 and reapply the template after your changes. We can update the RPM after you get a working version running.

On Wed, Oct 7, 2020, 8:35 AM James Cocker notifications@github.com wrote:

Thanks for adding this Anoop, but I'm struggling to get it to work. It may just be that the public root of a Silverstripe 4 site should be: {{ DOCUMENTROOT }}/public rather than just {{ DOCUMENTROOT }} (presume DOCUMENTROOT is public_html?)

I'm happy to refine and test this for you, by using a custom nginx.conf in the site root, but I keep getting Invalid/Require Reload, even when using the basic template from here: https://autom8n.com/docs/Application_template.html

So if you can help with a valid starting point, I'll do the rest!

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/AnoopAlias/AUTOM8N/issues/629#issuecomment-704904856, or unsubscribe https://github.com/notifications/unsubscribe-auth/AMLFPMIPDVB6KSEECBLCBODSJROBHANCNFSM4QDFSZJA .

AnoopAlias commented 4 years ago

@purplespider yes directly edit the template file and run

/opt/nDeploy/scripts/generate_config.py USER nginx -s reload

or you can simply test this by directly editing the generated config at

/etc/nginx/sites-enabled/domain.com.include nginx -s reload

If it's working, You can tell here or better send in a pull request for the change :)

purplespider commented 4 years ago

Thanks both. Unfortunately, I'm an end-user of a host using Autom8n, so I don't have that level of access.

Is there no way I can use the public_html/nginx.conf feature to help here?

AnoopAlias commented 4 years ago

@purplespider Unfortunately no .. because the root directive is already defined in the template

https://github.com/AnoopAlias/AUTOM8N/blob/ndeploy4/conf/5050.j2#L3

and nginx.conf would not allow a duplicate

But I am sure your host will understand this and ready to make the change if you open a support ticket with them. If they don't let me know and I will create a test environment to test this out with a SilverStripe v4 install.

Perhaps you can get the issues to link shared with the host, so they understand what's going on