* fix(error-handler): prevent silent() from re-enabling error handling during recovery
Root cause: silent() (core.func) unconditionally calls set -Eeuo pipefail
and trap 'error_handler' ERR after every command. When build_container()
intentionally disables error handling for its recovery section, any
intermediate call through silent()/ re-enables it. This causes the
grep/sed pipeline for missing_cmd extraction to trigger error_handler
(grep returns exit code 1 on no match + pipefail = fatal).
Fixes:
1. silent(): Save errexit state before disabling, only restore if it was
active. Callers that intentionally disabled error handling (e.g.
build_container recovery) are no longer silently re-enabled.
2. build.func: Add || true to missing_cmd grep pipeline as defense-in-depth
against pipeline failure propagation.
3. build.func: Add explicit set +Eeuo pipefail / trap - ERR after
post_update_to_api() call, before error classification grep/sed section.
4. build.func: Remove stale global combined_log variable from variables()
that used a different path format (/tmp/install-SESSION-combined.log)
than the actual local variable (/tmp/NSAPP-CTID-SESSION.log). The global
was never written to and caused confusion when error_handler displayed it.
* Update build.func
* chore(install): add Github source links to all setup_nodejs scripts
52 install scripts had a project website in '# Source:' but no GitHub
link. Merged the GitHub repo URL into the Source header as:
# Source: https://website.com/ | Github: https://github.com/OWNER/REPO
Repos sourced from fetch_and_deploy_gh_release calls, get_latest_github_release
calls, or known project repos for npm/pip installed apps.
Two scripts (fumadocs, pve-scripts-local) had no Source line at all —
added one. Shinobi skipped (GitLab-only, no GitHub repo).
* chore(install): add Github source links to all fetch_and_deploy scripts
77 additional install scripts had fetch_and_deploy_gh_release calls but
no GitHub link in the Source header. Merged the primary app repo into
the Source header as:
# Source: https://website.com/ | Github: https://github.com/OWNER/REPO
Where multiple fetch_and_deploy calls existed (app + dependency), the
primary app repo was selected:
- ersatztv: ErsatzTV/ErsatzTV (not ffmpeg)
- firefly: firefly-iii/firefly-iii (not data-importer)
- komga: gotson/komga (not kepubify dep)
- sabnzbd: sabnzbd/sabnzbd (not par2cmdline-turbo dep)
- signoz: SigNoz/signoz (not otel-collector)
- tunarr: chrisbenincasa/tunarr (not ffmpeg dep)
Also fixed cosmos-install.sh double https:// in Source URL.
Skipped: autocaliweb (source already on codeberg, GitHub repos are deps only)
* revert: restore misc/build.func and misc/core.func to main state
These error-handler fixes belong to fix/error-handler-recovery, not to
this sources-only branch.
* chore(ct,tools): sync Source headers with install/ and add Github links to addon scripts
The PHP_VERSION variable was only available within the setup_php
function call scope. By setting it separately before the function
call, it remains available for the nginx configuration heredoc.
Fixes#11439
The PHP_VERSION was passed inline to setup_php which uses it as a local
variable. This caused 'unbound variable' error in the nginx heredoc.
Setting PHP_VERSION before the function call keeps it in global scope
for use in the nginx configuration.
* fix(2fauth): use default PHP version instead of hardcoded 8.3
- Remove hardcoded PHP_VERSION=8.3, let setup_php use default (8.4)
- Remove invalid modules 'session' and 'openssl' (part of php-common)
- Remove duplicate 'cli' module (already in DEFAULT_MODULES)
- Make nginx fastcgi_pass use dynamic PHP_VERSION variable
Fixes installation on Debian 13 (Trixie) which ships PHP 8.4.
* Update 2fauth-install.sh
* feat(2fauth): ✨ Added 2FAuth
* refactor: 🚚 Changed path to user repo
* refactor: 🚚 Changed path to user repo
* refactor: 🚚 Changed path to user repo
* refactor: 🚚 Changed path to user repo
* refactor: 🚚 Changed path to user repo
* fix(2fauth): 🐛 Fixed path to build functions file
* fix(2fauth): 🐛 Fixed unbound variable
* fix(2fauth): 🐛 Use instead of for the directory name
* chore(2fauth): ✨ Added dependency package for improved composer performance
* chore(2fauth): ✨ Added dependency package as it's required
* chore(2fauth): ✨ Added dependency package `php8.2-fpm` as it's required
* fix(2fauth): 🐛 Fixed unbound variable
* fix(2fauth): 🐛 Fixed installation
* fix(install): 🐛 Fixed unassigned variable
* fix(install): 🐛 Fixed installation
* fix(install): 🐛 explicitly set ownership as last step
* revert: ⏪ Revert path rewrite to user repo
* revert: ⏪ Revert path rewrite to user repo
* refactor(2fauth): ⚰️ Removed commented-out code
* fix(2fauth): 🚚 Fixed path to remove correctly
* refactor(2fauth): 🎨 Changed from variables to static as requested
* docs(2fauth): 📝 Added notes for db credentials and the first account being an administrator account
* fix(2fauth): 🔊 Updated progress logging
* test(2fauth): 🚚 Changed pathes temporarily to user repo to test the App
* fix(2fauth): 🚑 Fixed wrong version file in update_script
* fix(2fauth): 💄 Removed duplicated version prefix v in messages
* Revert 'test(2fauth): 🚚 Changed pathes temporarily to user repo to test the App'
