Testing

[alexAubin]

Problem

• Install is broken for stupid reasons
• Fix #39
• Fix #34

Solution

• Add config before installing the .deb to avoid collabora stupidly looking for a cert file that doesn't exist
• Fix the start of the service (it wasnt finding the proper pattern in the service log because ... the service was already started...)
• Fix nginx config such that it doesn't just display nginx default page when reaching the domain ...
• Use ynh_add_config for clarity (in fact requires a fix in the core ... on the way ...)

I don't know how to fully test the app though (gotta integrate it in nextcloud i suppose)

[alexAubin]

Trying to investigate with a user why the nextcloud integration wasnt working, encountered this kind of issue https://github.com/CollaboraOnline/Docker-CODE/issues/49 ... turns out we don't have the <net> block config in our current loolwsd.xml

In fact, looking at the diff with the default loolwsd.xml we have quite a few missing things (probably because of lack of maintenance on this app)

c.f. attached file savagely extracted from https://www.collaboraoffice.com/repos/CollaboraOnline/CODE-debian10/loolwsd_6.4.3-3_amd64.deb :

loolwsd.xml.orig.txt

Diff with :

git diff --no-index loolwsd.xml.orig.txt loolwsd.xml

[Thatoo]

Does it work simply replacing the current yunohost loolwsd.xml file by the loolwsd.xml.orig.txt you extracted?

[alexAubin]

I don't know

[ericgaspar]

!testme

[yunohost-bot]

:rocket:

[ericgaspar]

Backup/restore tests seem to be successful in the old CI ...

[alexAubin]

Zblerg, not sure what happens from the log, there's nothing apart from a "Killed" in the systemd log, which could suggest that it's a resource issue, or that the process got killed by systemd after a "stop" being triggered ...

[alexAubin]

I savagely restarted the test just in case it's a tmp issue

[ericgaspar]

@alexAubin ping for merge?

[alexAubin]

We still need to address the previous comment https://github.com/YunoHost-Apps/collabora_ynh/pull/40#issuecomment-760370748 ... Like, I was discussing with a user the day I wrote it that could not integrate Collabora in Nextcloud because of the lack of some section of this config file .. But my todolist is too huge to have time to look deeper into this :/

[ericgaspar]

!testme

[yunohost-bot]

:v: