contao / contao-manager

Contao Manager
GNU Lesser General Public License v3.0
85 stars 33 forks source link

Directory Setup breaks Install if directory is already properly configured #647

Closed asconsulting closed 3 years ago

asconsulting commented 3 years ago

Screenshot_2021-04-22 Contao Manager dev-dws brightcloudstudioserver com

There seems to be no way to skip this step if the directory is already setup properly. As you can see the directory structure is already setup and /web is set as the web root with Contao installing to public_html, and world readable files living in /web. Ideally there should be a way to skip this step if everything is already setup.

Related, is the empty directory warning. On cpanel hosts for example, AutoSSL creates .well-known folder for Let's Encrypt validation. It would be nice to also be able to ignore the directory not being empty messages as well. If you know what you are doing and have already customized your environment.

asconsulting commented 3 years ago

So if the directory structure is indeed empty all the way up to the /web directory you can get around the first step. However it would be nice to be able to ignore the empty directory warning because of the AutoSSL creating the validation file. This is frequently out of the control of the person installing Contao on cPanel hosts since those directories are created automatically and will respawn if deleted.

aschempp commented 3 years ago

I have improved the Contao Manager to ignore the .well-known directory if present in 1f0073fb616e47868525eeb38ba943618edc7224. I hope this fixed your issue 😊