fix(miniflux): use systemctl to check service instead of file path

The miniflux.service file is installed by the deb package at /usr/lib/systemd/system/ not /etc/systemd/system/, causing the update script to fail with 'No Miniflux Installation Found!'. Using 'systemctl is-enabled' is more robust and works regardless of where the service file is located.
fix(2fauth): set PHP_VERSION globally for nginx config
2025-12-16 20:23:32 +01:00 · 2025-12-16 11:08:25 +01:00 · 2025-12-16 11:01:10 +01:00 · 2025-12-16 09:46:29 +00:00 · 2025-12-16 10:46:07 +01:00
3 changed files with 9 additions and 2 deletions
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@ -12,6 +12,12 @@ Exercise vigilance regarding copycat or coat-tailing sites that seek to exploit

 ## 2025-12-16

+### 🚀 Updated Scripts
+
+  - #### 🐞 Bug Fixes
+
+    - 2fauth: bump to php8.4 [@MickLesk](https://github.com/MickLesk) ([#10019](https://github.com/community-scripts/ProxmoxVE/pull/10019))
+
 ## 2025-12-15

 ### 🆕 New Scripts
--- a/ct/miniflux.sh
+++ b/ct/miniflux.sh
@ -23,7 +23,7 @@ function update_script() {
  header_info
  check_container_storage
  check_container_resources
-  if [[ ! -f /etc/systemd/system/miniflux.service ]]; then
+  if ! systemctl -q is-enabled miniflux 2>/dev/null; then
    msg_error "No ${APP} Installation Found!"
    exit
  fi
--- a/install/2fauth-install.sh
+++ b/install/2fauth-install.sh
@ -17,7 +17,8 @@ msg_info "Installing Dependencies"
 $STD apt install -y nginx
 msg_ok "Installed Dependencies"

-PHP_MODULE="common,ctype,fileinfo,mysql,tokenizer,dom,redis" PHP_VERSION="8.4" PHP_FPM="YES" setup_php
+PHP_VERSION="8.4"
+PHP_MODULE="common,ctype,fileinfo,mysql,tokenizer,dom,redis" PHP_FPM="YES" setup_php
 setup_composer
 setup_mariadb
 MARIADB_DB_NAME="2fauth_db" MARIADB_DB_USER="2fauth" setup_mariadb_db
Author	SHA1	Message	Date
CanbiZ	bde63a9c7b	fix(miniflux): use systemctl to check service instead of file path The miniflux.service file is installed by the deb package at /usr/lib/systemd/system/ not /etc/systemd/system/, causing the update script to fail with 'No Miniflux Installation Found!'. Using 'systemctl is-enabled' is more robust and works regardless of where the service file is located.	2025-12-16 11:08:25 +01:00
CanbiZ	51ba8ca7c8	fix(2fauth): set PHP_VERSION globally for nginx config 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.	2025-12-16 11:01:10 +01:00
community-scripts-pr-app[bot]	6d78e4da6b	Update CHANGELOG.md (#10022 ) Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>	2025-12-16 09:46:29 +00:00
CanbiZ	a3fb28105d	2fauth: bump to php8.4 (#10019 ) * 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	2025-12-16 10:46:07 +01:00