search

extdn / installer-m2

Universal extension installer for Magento 2, see introductory blog post here
https://fooman.com/blog/introducing-the-extdn-installer-for-magento-2.html
MIT License
24 stars 0 forks source link

What about running as user `root`? #2

Closed jissereitsma closed 5 years ago

jissereitsma commented 5 years ago

Should the installer deny running as user root or file owner root (ID 0)?

fooman commented 5 years ago

In an ideal world no Magento installation should be owned by the root user. Unfortunately I have come across enough to know that the ideal does not always happen.

The current check that is performed is against existing files and if there is a mismatch will exit. This should cover scenarios where we are executing as root when we shouldn't.