Skip to content

Conversation

ElderBlade
Copy link

This change by JMarcosHP fixes OnlyOffice connection error when run behind nginx proxy manager, reference issue #766

@mickael-kerjean
Copy link
Owner

The PR would break the default setup where the ONLY OFFICE setup come from this and is set to http://onlyoffice/, aka a URL that's not available to the frontend

@JMarcosHP
Copy link

Can there be another alternative?
We need support for custom domains or a way to expose the internal onlyoffice URL for filestash.

@JMarcosHP
Copy link

Or another env variable for external URL that overrides ONLYOFFICE_URL if set.

@mickael-kerjean
Copy link
Owner

mickael-kerjean commented Nov 13, 2024

Or another env variable for external URL that overrides ONLYOFFICE_URL if set.

That wouldn't be great from a end user point of view and that approach will stop working in the next few months. To understand why, we need to dig through how this plugin was done in the first place and why it was made the way it currently is. The general model of this plugin is to proxy the traffic to only office. This proxy approach was taken initially because:

  1. not breaking the global Content Security Policy would be either impossible or an absolute nightmare. Sure the suggested approach currently work because of a temporary dirty hack which will get rolled back once the migration to the new frontend is done. Once that's reenabled, it won't be possible for a third party app to inject code like this and that's a good thing from a security point of view
  2. I didn't want to force anyone to expose only office to the internet. When I see the cheer number of people using Filestash without bothering with SSL, having to create multiple certificates make things potentially a lot more tricky to install.
  3. it gives a way to block traffic to only office if a vulnerability is found

Can there be another alternative?

yes, the best approach is to address the root cause of the issue which is a bug in here.

@mickael-kerjean mickael-kerjean force-pushed the master branch 6 times, most recently from b95fa04 to 11c4a74 Compare September 2, 2025 13:52
@mickael-kerjean mickael-kerjean force-pushed the master branch 5 times, most recently from 9757ff1 to 7fa253f Compare September 11, 2025 05:00
@mickael-kerjean mickael-kerjean force-pushed the master branch 2 times, most recently from 5e9f59a to b9f2ee9 Compare September 23, 2025 04:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants