search

sergejmueller / wpcheck

Vulnerability scanner for WordPress based on Node.js
MIT License
64 stars 11 forks source link

Don't use root permissions #23

Closed HeikoMamerow closed 8 years ago

HeikoMamerow commented 8 years ago

Hi Sergej, a properly installed node/npm does not required root permissions. Therefore you should remove sudo from your install example.

bad: sudo npm install ...
good: npm install ...

Tips for fixing npm permissions: https://docs.npmjs.com/getting-started/fixing-npm-permissions

swissspidy commented 8 years ago

This article is helpful as well: http://blog.michaelarestad.com/2015/08/24/how-to-unfuck-permissions/

sergejmueller commented 8 years ago

Sorry for delay, guys. It's holiday time ;)

You right. Some time ago I had it without sudo too. Then I saw how other packages document the install -g command.

I will remove the [sudo] prefix from the install line and add the two links to the readme to reduce user requests (aka support). Thanks.