YunoHost-Apps / onlyoffice_ynh

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

OO won't start after upgrade from 7.5 to 8.0 - 502 error with missing parameters #126

Open Jaxom99 opened 4 months ago

Jaxom99 commented 4 months ago

Describe the bug

Onlyoffice returns 502 errors after upgrading.

Context

Steps to reproduce

1° upgrade tthrough CLI 2° errors 502 on web interface 3° the docserver logs shows an error because a setting is missing : wopi.dummy.sampleFilePath 4° I add the settings manually in /etc/onlyoffice/documentserver.default.json to a random location /tmp/wopi 5° after service restart, same error with setting wopi.dummy.enable 6° After adding this setting as 'false', the restart works alright.

Expected behavior

Upgrade from yunohost should take care about missing settings.

Logs

upgrade successful : https://paste.yunohost.org/raw/ewazumavub Example of error messages :

/snapshot/server/DocService/node_modules/config/lib/config.js:203
    throw new Error('Configuration property "' + property + '" is not defined');
    ^

Error: Configuration property "wopi.dummy.enable" is not defined
    at Config.get (/snapshot/server/DocService/node_modules/config/lib/config.js:203:11)
    at Object.<anonymous> (/snapshot/server/DocService/sources/server.js)
    at Module._compile (pkg/prelude/bootstrap.js:1930:22)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1159:10)
    at Module.load (node:internal/modules/cjs/loader:981:32)
    at Function.Module._load (node:internal/modules/cjs/loader:822:12)
    at Function.runMain (pkg/prelude/bootstrap.js:1983:12)
    at node:internal/main/run_main_module:17:47
Quiwy commented 4 months ago

Same problem here. Just added

    "dummy": {
        "enable": true,
        "sampleFilePath": ""
    }

as described in the onlyoffice doc, and working after service restart.