Add OpenCloud connection provider - #42
Open
palchrb wants to merge 1 commit into
Open
Conversation
OpenCloud (opencloud.eu) serves its WOPI-based web office apps on /external-<app> editor URLs and answers the ownCloud-compatible status.php probe, so a data-driven provider entry is all the connect dialog needs: reachability check via status.php, editor tabs promoted by the /external-* URL pattern, and a monochrome OpenCloud mark for the connections list in both themes. The editorPage regex matches any /external-<app> page instead of hardcoding an app name, because the WOPI app name on the server (COLLABORATION_APP_NAME) is deployment configuration, not a constant. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01VzBsHpG5rHzxwgRofBDnw5
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Adds OpenCloud (https://opencloud.eu) as a cloud provider in the connect
dialog — a data-driven entry under
common/loginpage/providers/opencloud/:config.jsonplus list/button icons for both themes (monochrome, derived fromOpenCloud's official mark). Five new files, no code or build changes — the
providers directory is enumerated at app startup.
This answers the community-integration invitation in Euro-Office/core#77:
OpenCloud pairs naturally with the Euro-Office document server, since OpenCloud
serves its web office through WOPI natively — no platform connector app needed.
Design notes
check.urlisstatus.php: OpenCloud serves the ownCloud-compatible statusendpoint, so reachability probing works like the ownCloud provider's.
editorPageisregex:\/external-[a-z0-9-]+rather than a hardcoded appname: OpenCloud opens WOPI apps on
/external-<app>URLs where<app>isthe server's configured
COLLABORATION_APP_NAME— deployment configuration,not a constant.
editorPage(/apps/onlyoffice/) is the classic connector-app path andnever matches, so documents are never promoted to editor tabs.
editorFrameSize— the default behaves correctly with OpenCloud'seditor page.
How this was tested
Against a real OpenCloud 7.5 instance (external reverse proxy, external OIDC
IdP) with the provider directory dropped into an installed DesktopEditors
build's
providers/folder (runtime enumeration — no rebuild needed):/external-*match;editing, saving and co-editing with a browser user work
portal:login(proposed at opencloud-eu/web-extensions#), the portal appears and
persists in the Clouds list
Ref: Euro-Office/core#77