Page MenuHomePhorge

Make it possible to configure collabora with internal networking
ClosedPublic

Authored by mollekopf on Dec 26 2023, 8:39 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 23, 3:21 PM
Unknown Object (File)
Sun, Oct 20, 11:50 PM
Unknown Object (File)
Sep 13 2024, 8:27 AM
Unknown Object (File)
Sep 11 2024, 5:25 PM
Unknown Object (File)
Sep 8 2024, 5:37 AM
Unknown Object (File)
Sep 8 2024, 4:58 AM
Unknown Object (File)
Sep 4 2024, 2:19 AM
Unknown Object (File)
Sep 4 2024, 12:21 AM
Subscribers

Details

Summary

We need some extra overrides if collabora and chwala are not reachable
via the same domain internally and externally. This allows us to use
cluster-internal networking for all direct connections.

Diff Detail

Repository
rC chwala
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

mollekopf created this revision.
mollekopf added a reviewer: Restricted Project.Dec 26 2023, 8:39 AM
machniak subscribed.

If I'm not mistaken fileapi_wopi_office is already internal (chwala to collabora) connection, no need for *_server option. Am I wrong? As for file_api_server_url, it's from kolab_files plugin. I think it would be good to create and document a separate Chwala option.

This revision now requires changes to proceed.Dec 26 2023, 9:01 AM
This revision was not accepted when it landed; it landed in state Needs Revision.Dec 26 2023, 9:19 AM
This revision was automatically updated to reflect the committed changes.

This has not actually landed.

If I'm not mistaken fileapi_wopi_office is already internal (chwala to collabora) connection, no need for *_server option. Am I wrong?

It seems you're right, I think I got confused by a comment in the config file.

As for file_api_server_url, it's from kolab_files plugin. I think it would be good to create and document a separate Chwala option.

The kolab_files option is kolab_files_server_url (I think?), but I followed the same pattern as kolab_files_url/kolab_files_server_url.

Removed the useless WOPI_OFFICE server variant,
adjusted the default config file to include the new option and remove the misleading note.

This revision is now accepted and ready to land.Dec 28 2023, 9:16 AM