Skip to content

[pull] main from nextcloud:main #26

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.

Sign up for GitHub

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
pull merged 2 commits into from
Dec 29, 2025
Merged

[pull] main from nextcloud:main #26

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
cb927ae
Change rm command to use -f option
enoch85 Dec 28, 2025
e4fee90
Update AppAPI deployment method descriptions
enoch85 Dec 28, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
19 changes: 10 additions & 9 deletions addons/appapi.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ AppAPI is required to run External Apps (ExApps) which are containerized applica
that extend Nextcloud's capabilities, particularly AI applications.

This script supports two deployment methods:
1. HaRP (recommended for NC 32+) - Modern deployment with reverse proxy
1. HaRP - Modern deployment with reverse proxy
2. Direct Docker Socket - Simple setup for local installations

If you don't plan to use External Apps, you can run this script again to disable AppAPI"
Expand Down Expand Up @@ -228,20 +228,21 @@ fi
choice=$(whiptail --title "$TITLE" --menu \
"Choose a deployment method for AppAPI:

HaRP (Recommended for NC 32+):
• Simplest setup with best performance
HaRP (Requires public domain):
• Requires reverse proxy configuration and a publicly accessible Nextcloud URL
• Best performance
• Nextcloud 32+ only!
• Direct communication between browser and ExApps
• Built-in brute-force protection
• Requires reverse proxy configuration and a publicly accessible Nextcloud URL

Docker Socket (Simple):
• Direct Docker socket access
Docker Socket (Local installations):
• Good for local-only installations without external access
• Direct Docker socket access
• No additional containers needed
• Less secure than HaRP

${GPU_INFO:-No GPU detected - CPU will be used}" "$WT_HEIGHT" "$WT_WIDTH" 4 \
"HaRP" "(Recommended) Modern proxy-based deployment" \
"HaRP" "Modern proxy-based deployment" \
"Docker Socket" "Direct socket access (simpler)" \
"Cancel" "Exit without changes" 3>&1 1>&2 2>&3)

Expand Down Expand Up @@ -642,7 +643,7 @@ if [ "$DEPLOY_METHOD" = "harp" ]
then
msg_box "Congratulations! $SCRIPT_NAME was successfully configured with HaRP!

Deployment Method: HaRP (Recommended)
Deployment Method: HaRP
Daemon Name: $DAEMON_NAME
Compute Device: ${COMPUTE_DEVICE^^}
HaRP Container: $HARP_CONTAINER_NAME
Expand Down Expand Up @@ -695,4 +696,4 @@ consider switching to HaRP for better security and performance.

Documentation:
https://docs.nextcloud.com/server/latest/admin_manual/exapps_management/"
fi
fi
2 changes: 1 addition & 1 deletion static/change-ncadmin-profile.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@ source /var/scripts/fetch_lib.sh
DEBUG=0
debug_mode

rm "/home/$UNIXUSER/.profile"
rm -f "/home/$UNIXUSER/.profile"

cat <<-UNIXUSER-PROFILE > "$UNIXUSER_PROFILE"
# ~/.profile: executed by the command interpreter for login shells.
Expand Down