countly-server runs as root

[milgner]

The supervisord configuration specifies user=root which in itself is doubtful whether it's a good idea but could be tolerated.
But then, the application user isn't overriden in neither program:countly-dashboard or program:countly-api sections.
Right now I don't have access to Ubuntu or RHEL systems to update the installer scripts but as I'm currently modifying the configuration files in this regard, I'd be willing to contribute a PR if you like.
Do you have a CI system to test these changes on your target platforms?

[derekmoorecountly]

Agreed - there could be a need to check that for other services as well.. If you need, we can create two servers (RHEL/Ubuntu) for you so you can make your tests and send a PR.

[milgner]

The next few weeks I'll be quite busy deploying our app and website, but afterwards I'll investigate setting up a few Vagrant-based scripts to run some automated tests in Ubuntu and RHEL. Since I'm on Debian right now, I went through the setup manually and didn't look at the automated install scripts yet.
On another note, thanks for a very useful piece of software! 👍

[derekmoorecountly]

Many thanks for your interest 💯

[milgner]

There's a fix in my feature branch milgner/countly-server@db971f7. Should I wait for #103 to be closed or should I push my changes to the same branch? It's not much code after all but I usually prefer to keep pull requests topical.

[ar2rsawseen]

Based on provided example implemented installing countly as another user here: 89c950d

So there are now two scripts

countly.install.sh which installs as root
countly.user.install.sh which installs as countly user

You still need to run install script as root though