[Bug 246623] [NEW PORT] www/caddy2 (Update to www/caddy)

[bugzilla-noreply at freebsd.org]

https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=246623

--- Comment #23 from Basil Hendroff <basil.hendroff at gmail.com> ---
Hi Daniel

An amendment to the etc/rc.d/caddy initialisation script appears to be
necessary to address an issue that has been identified. For the details, please
refer to
https://caddy.community/t/how-to-use-the-expanded-form-of-php-fastcgi/9228. 

Your proposed caddy rc.d script was used as the basis for the caddy script
located at
https://github.com/danb35/freenas-iocage-nextcloud/blob/caddyv2/includes/caddy.
It includes a suggested fix for the issue described as well as other
refinements you may wish to consider. These are outlined below:

1. start_cmd redirects stdout and stderr to a log file (default is
/var/log/caddy.log) to address the issue identifed above. It also serves to
eliminate 'console spam'
2. pidfile was defined, but not used in the master copy. It is actively used in
this copy.
3. required files - Enables a check for the existence of the Caddy binary and
Caddyfile. This is a built-in check. With the check, a service caddy start will
alert the user if either of these files is missing; without it, no cue is
provided.
4. Superfluous command definitions - stop and restart, as well as many other
commands such as status and extracommands, are automatically supported as
defined in rc(8). See
https://www.freebsd.org/cgi/man.cgi?query=rc&sektion=8&manpath=freebsd-release-ports.
There doesn't appear to be any added value in including these command
definitions. If anything, it makes things a little more ambiguous as it
detracts from the commands that must be defined or altered i.e. start, validate
and reload.

-- 
You are receiving this mail because:
You are the assignee for the bug.