From 45fb7202ac0e606ccb7b4fe95f169424f0a6cabc Mon Sep 17 00:00:00 2001
From: Carl George <carl@george.computer>
Date: Mon, 5 Apr 2021 15:01:20 -0500
Subject: notify: Send all sd_notify signals from main caddy process (#4060)

Initial sd_notify support was added in #3963, but that sent signals from
both cmdRun and cmdReload.  This approach has two drawbacks:

- Reloads initiated via the API do not send signals.
- The signals are sent from different processes, which requires the
  `NotifyAccess=exec` directive in the unit file.

This change moves the NotifyReloading and NotifyReadiness invocations to
Load, which address both of those drawbacks.  It also adds a
complimentary NotifyStopping method which is invoked from handleStop.
All the notify methods are defined in a notify package to avoid an
import loop.
---
 cmd/notify_linux.go | 68 -----------------------------------------------------
 1 file changed, 68 deletions(-)
 delete mode 100644 cmd/notify_linux.go

(limited to 'cmd/notify_linux.go')

diff --git a/cmd/notify_linux.go b/cmd/notify_linux.go
deleted file mode 100644
index 924c00f..0000000
--- a/cmd/notify_linux.go
+++ /dev/null
@@ -1,68 +0,0 @@
-// Copyright 2015 Matthew Holt and The Caddy Authors
-//
-// Licensed under the Apache License, Version 2.0 (the "License");
-// you may not use this file except in compliance with the License.
-// You may obtain a copy of the License at
-//
-//     http://www.apache.org/licenses/LICENSE-2.0
-//
-// Unless required by applicable law or agreed to in writing, software
-// distributed under the License is distributed on an "AS IS" BASIS,
-// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-// See the License for the specific language governing permissions and
-// limitations under the License.
-
-package caddycmd
-
-import (
-	"io"
-	"net"
-	"os"
-	"strings"
-)
-
-// The documentation about this IPC protocol is available here:
-// https://www.freedesktop.org/software/systemd/man/sd_notify.html
-
-func sdNotify(path, payload string) error {
-	socketAddr := &net.UnixAddr{
-		Name: path,
-		Net:  "unixgram",
-	}
-
-	conn, err := net.DialUnix(socketAddr.Net, nil, socketAddr)
-	if err != nil {
-		return err
-	}
-	defer conn.Close()
-
-	if _, err := io.Copy(conn, strings.NewReader(payload)); err != nil {
-		return err
-	}
-	return nil
-}
-
-// notifyReadiness notifies systemd caddy that has finished its
-// initialization routines.
-func notifyReadiness() error {
-	val, ok := os.LookupEnv("NOTIFY_SOCKET")
-	if !ok || val == "" {
-		return nil
-	}
-	if err := sdNotify(val, "READY=1"); err != nil {
-		return err
-	}
-	return nil
-}
-
-// notifyReadiness notifies systemd that caddy is reloading its config.
-func notifyReloading() error {
-	val, ok := os.LookupEnv("NOTIFY_SOCKET")
-	if !ok || val == "" {
-		return nil
-	}
-	if err := sdNotify(val, "RELOADING=1"); err != nil {
-		return err
-	}
-	return nil
-}
-- 
cgit v1.2.3