Skip to content

Commit a59e25c

Browse files
jaapmarcusKristan Kenney
andauthored
Update conf only after upgrade / reload from backup (hestiacp#3908)
* Update conf only after upgrade / reload from backup * Fix typo in function call --------- Co-authored-by: Kristan Kenney <139469540+krismkenn@users.noreply.github.com>
1 parent d7cfe44 commit a59e25c

File tree

4 files changed

+22
-20
lines changed

4 files changed

+22
-20
lines changed

bin/v-add-sys-ip

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -201,7 +201,8 @@ fi
201201

202202
# Adapt Port Listing in HESTIA NGINX Backend
203203
syshealth_adapt_hestia_nginx_listen_ports
204-
204+
# Also update resolver
205+
syshealth_adapt_nginx_resolver
205206
#----------------------------------------------------------#
206207
# Hestia #
207208
#----------------------------------------------------------#

func/syshealth.sh

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -623,3 +623,22 @@ function syshealth_adapt_hestia_nginx_listen_ports() {
623623
[ $? -ne 0 ] && NGINX_BCONF_CHANGED="yes"
624624
rm -f "$NGINX_BCONF_TEMP" > /dev/null 2>&1
625625
}
626+
627+
syshealth_adapt_nginx_resolver() {
628+
NGINX_CONF="/usr/local/hestia/nginx/conf/nginx.conf"
629+
if grep -qw "1.0.0.1 8.8.4.4 1.1.1.1 8.8.8.8" "$NGINX_CONF"; then
630+
for nameserver in $(grep -is '^nameserver' /etc/resolv.conf | cut -d' ' -f2 | tr '\r\n' ' ' | xargs); do
631+
if echo "$nameserver" | grep -Pq "^(\d{1,3}\.){3}\d{1,3}$"; then
632+
if [ -z "$resolver" ]; then
633+
resolver="$nameserver"
634+
else
635+
resolver="$resolver $nameserver"
636+
fi
637+
fi
638+
done
639+
640+
if [ -n "$resolver" ]; then
641+
sed -i "s/1.0.0.1 8.8.4.4 1.1.1.1 8.8.8.8/$resolver/g" "$NGINX_CONF"
642+
fi
643+
fi
644+
}

src/deb/nginx/hestia

Lines changed: 0 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -31,26 +31,7 @@ set -e
3131

3232
. /etc/profile.d/hestia.sh
3333

34-
update_nginx_resolver() {
35-
if grep -qw "1.0.0.1 8.8.4.4 1.1.1.1 8.8.8.8" "$NGINX_CONF"; then
36-
for nameserver in $(grep -is '^nameserver' /etc/resolv.conf | cut -d' ' -f2 | tr '\r\n' ' ' | xargs); do
37-
if echo "$nameserver" | grep -Pq "^(\d{1,3}\.){3}\d{1,3}$"; then
38-
if [ -z "$resolver" ]; then
39-
resolver="$nameserver"
40-
else
41-
resolver="$resolver $nameserver"
42-
fi
43-
fi
44-
done
45-
46-
if [ -n "$resolver" ]; then
47-
sed -i "s/1.0.0.1 8.8.4.4 1.1.1.1 8.8.8.8/$resolver/g" "$NGINX_CONF"
48-
fi
49-
fi
50-
}
51-
5234
start_nginx() {
53-
update_nginx_resolver
5435
start-stop-daemon --start --quiet --pidfile $NGINX_PID \
5536
--retry 5 --exec $NGINX_DAEMON --oknodo
5637
}

src/deb/nginx/postinst

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -13,6 +13,7 @@ invoke-rc.d hestia start || true
1313
# Adapt Port Listing in HESTIA NGINX Backend
1414
source /usr/local/hestia/func/syshealth.sh # Load syshealth functions
1515
syshealth_adapt_hestia_nginx_listen_ports # Adapt port listing
16+
syshealth_adapt_nginx_resolver
1617

1718
# Run triggers only on updates
1819
if [ ! -e "/usr/local/hestia/data/users/" ]; then

0 commit comments

Comments
 (0)