YunoHost-Apps / onlyoffice_ynh

OnlyOffice package for YunoHost
https://www.onlyoffice.com
GNU General Public License v3.0
19 stars 19 forks source link

Yunohost install leaves default NodeJS port, which causes conflicts. #78

Open czen opened 2 years ago

czen commented 2 years ago

Install script doesn't setup port for internal NodeJS server.

NodeJS server for onlyoffice document server is setup via supervisord with config in /etc/supervisor/conf.d/ds-docservice.conf Config for NodeJS server is in /etc/onlyoffice/documentserver/default.json There is a port setting in this default.json The only command to change app ports in current Yunohost OnlyOffice install script is this:

echo onlyoffice-documentserver onlyoffice/ds-port select $port | debconf-set-selections

This sets up the port that nginx would listen on to serve static content and reverse proxy to the NodeJS server.

If you only setup the nginx port and leave default 8000 for NodeJS, 8000 would conflict with some other app on the system or with app installed via Yunohost. NodeJS server won't start, Only welcome page would work.

This package was public for 2 years at this point. How would nobody stumble on this obvious problem is beyond my understanding. Did no one ever try to install document server alongside any other packages?