Specify HOST for express server to listen on

[DerOetzi]

New pull request for feature request #168

[digitaldan]

how about something like this, and remove the line setting this on the app?

  var server_params = {
            port :  process.env.PORT || 3000
        }
        
        if(process.env.HOST){
            server_params.host = process.env.HOST;
        }

[DerOetzi]

I do not see any pros and cons to this change unless the configuration is needed elsewhere in the application. In addition, the possibility of setting a default value is lost.

[DerOetzi]

In addition, you can not view the configuration in the log

[DerOetzi]

I have resolved some merge conflicts to master branch. Now it is possible to merge this pull requests.

[marziman]

@DerOetzi is this still relevant? I see no problem with this PR.

[DerOetzi]

For me personally this isn't relevant anymore as I don't use openhab-cloud but using a vpn connection behind a haproxy setup. But I think for server admins which want there own cloud instance on a server with multiple IPs this is still a good feature.

[electraport]

Sorry for the thread necromancy but since it's still open, I have noticed after a batch of test installs that the server seems to come up listening on IPv6 only, and requires something similar to:

var server = app.listen(system.getNodeProcessPort(), '0.0.0.0', function () {
    logger.info('openHAB-cloud: express server listening on port ' + system.getNodeProcessPort());

in /app.js to listen on IPv4 - where I added the comma and '0.0.0,0' between system.getNodeProcessPort and the callback.

I can see the arguments for the ability to set a default in e.g. /config.json and am happy to put up a pr for this if you want it.