-
Notifications
You must be signed in to change notification settings - Fork 1
Add HaRP support for Nextcloud 32+ #6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Conversation
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
Copilot
AI
changed the title
[WIP] Add HaRP support for workflow_ocr_backend
Add HaRP support for Nextcloud 32+
Oct 27, 2025
R0Wi
requested changes
Oct 27, 2025
e6f035b to
7835aca
Compare
Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>
Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>
Co-authored-by: R0Wi <19730957+R0Wi@users.noreply.github.com>
* Add docker socket to devcontainer * Consolidate chmod +x for start.sh
7835aca to
b9582cf
Compare
* Ensure containers "serviceuser" is able to write /frpc.toml * Write python integration tests to ensure container is able to start properly with frpc enabled
b8e31c0 to
528f684
Compare
528f684 to
8f092d8
Compare
* Switch to "gosu" to start the main process as "serviceuser" instead of using docker "USER" directive * Default user for docker commands will be root now (so NC docker commands are able to copy certs etc) * Let serviceuser own /certs directory so that frpc process is able to use the certs from there
R0Wi
added a commit
that referenced
this pull request
Nov 16, 2025
…arp-support Add HaRP support for Nextcloud 32+
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.
Nextcloud 32 recommends HaRP (AppAPI HaProxy Reversed Proxy) over Docker Socket Proxy for ExApp deployments. This PR implements HaRP support while maintaining full backward compatibility with DSP.
Changes
Core Implementation
HP_SHARED_KEYis present, passes through to normal startup otherwiseapk add frp), addcurlandbashdependencies, update ENTRYPOINT to use start.sh wrapperDocumentation
Compatibility
Works transparently with both deployment methods:
TLS automatically enabled when
/certs/frpdirectory is present.Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.