littlebizzy / slickstack

Lightning-fast WordPress on Nginx
https://slickstack.io
GNU General Public License v3.0
629 stars 112 forks source link

ss-perms - touch #48

Closed Bronislawsky closed 4 years ago

Bronislawsky commented 4 years ago

in ss-perms ss-perms:96:mkdir touch /var/www/meta/.ssh

I beleive mkdir should be removed

jessuppi commented 4 years ago

@Bronislawsky Thanks this is now fixed.

/var/www/meta/.ssh is a directory, so needs mkdir only (but not touch) to ensure the parent directory of /var/www/meta/.ssh/authorized_keys exists.

Ref: https://github.com/littlebizzy/slickstack/commit/4bb0bc7b5db706031bb49cb32f6c04706e41e1e9

Some of this might be cleaned up to be more efficient in the future, in any case.

Related Issue: https://github.com/littlebizzy/slickstack/issues/46