Created attachment 242626 [details] appjail-2.5.2.patch Description: ChangeLog: https://github.com/DtxdF/AppJail/releases/tag/v2.5.2 * Fixed: invalid lib_check_ipv4addr function name when using the address option in quick:virtualnet. * Fixed: missing empty variables in *_set_[expose:proto|virtualnet:interface_desc] (after calling a function and defining a variable, the variable is not emptied for the next call, the value is reused when the parameter is not called again, resulting in an incorrect configuration). * Added: template check before use it in quick. * Added: check for when the jail is gone in healthcheck. * Added: -Q parameter in libexec/appjail-config/tok. * Fixed: load of a kernel module at startup (at startup lib_check_kernmod does not correctly detect whether a kernel module is loaded or not, so kldload -n is used to load a kernel module if it is not already loaded). * Added: missing check for when a template does not exist in quick. * Fixed: race condition when creating the bridges at startup. * Added: removing of a Carriege Feed character when a Makejail is included (although the format requires Line-Feed to be the new line it is worthwhile for some users to remove Carriege Feed when a Makejail is included). * Fixed: CMD usage (usage shows that cmd is optional when it is not). * Fixed: CMD with missing arguments. * Fixed: incorrect token count in CMD. * Fixed: incorrect variable name appjail_logfile -> appjail_natnet_logfile in etc/rc.d/appjail-natnet. * Fixed: race condition in expose & nat. * Changed: apt-get to apt in jail & pkg. * Added: "Support" section in README. * Added: support for line-continuation in Makejails. * Improved: CMD parsing (CMD has been improved when parsing the shell command by not doing difficult and error-prone things that can result in unexpected behavior. CMD now passes host variables to the jail using env(1) which is a bit slow but more reliable than simply passing the variable carelessly. In addition, CMD can execute virtually any shell command without manually escaping harmful characters). * Added: --make-arg-env parameter in VAR. * Removed: parsing the variable as a shell command in VAR (shell-like parsing allows a friendlier syntax when passing build-args from the command-line, but does not allow many characters such as ", so the ugly version is preferred). QA: * portlint: OK (looks fine.) * testport: OK (poudriere: 13.2-RELEASE, amd64, DEBOOTSTRAP DOAS DOCS GIT tested)
Committed, Thanks!
A commit in branch main references this bug: URL: https://cgit.FreeBSD.org/ports/commit/?id=116f843a819fae51c68d95e09607d674e1c0a3ef commit 116f843a819fae51c68d95e09607d674e1c0a3ef Author: Jesús Daniel Colmenares Oviedo <DtxdF@disroot.org> AuthorDate: 2023-06-06 06:47:21 +0000 Commit: Fernando Apesteguía <fernape@FreeBSD.org> CommitDate: 2023-06-06 15:06:56 +0000 sysutils/appjail: Update to 2.5.2 ChangeLog: https://github.com/DtxdF/AppJail/releases/tag/v2.5.2 * Fixed: invalid lib_check_ipv4addr function name when using the address option in quick:virtualnet. * Fixed: missing empty variables in *_set_[expose:proto|virtualnet:interface_desc] (after calling a function and defining a variable, the variable is not emptied for the next call, the value is reused when the parameter is not called again, resulting in an incorrect configuration). * Added: template check before use it in quick. * Added: check for when the jail is gone in healthcheck. * Added: -Q parameter in libexec/appjail-config/tok. * Fixed: load of a kernel module at startup (at startup lib_check_kernmod does not correctly detect whether a kernel module is loaded or not, so kldload -n is used to load a kernel module if it is not already loaded). * Added: missing check for when a template does not exist in quick. * Fixed: race condition when creating the bridges at startup. * Added: removing of a Carriege Feed character when a Makejail is included (although the format requires Line-Feed to be the new line it is worthwhile for some users to remove Carriege Feed when a Makejail is included). * Fixed: CMD usage (usage shows that cmd is optional when it is not). * Fixed: CMD with missing arguments. * Fixed: incorrect token count in CMD. * Fixed: incorrect variable name appjail_logfile -> appjail_natnet_logfile in etc/rc.d/appjail-natnet. * Fixed: race condition in expose & nat. * Changed: apt-get to apt in jail & pkg. * Added: "Support" section in README. * Added: support for line-continuation in Makejails. * Improved: CMD parsing (CMD has been improved when parsing the shell command by not doing difficult and error-prone things that can result in unexpected behavior. CMD now passes host variables to the jail using env(1) which is a bit slow but more reliable than simply passing the variable carelessly. In addition, CMD can execute virtually any shell command without manually escaping harmful characters). * Added: --make-arg-env parameter in VAR. * Removed: parsing the variable as a shell command in VAR (shell-like parsing allows a friendlier syntax when passing build-args from the command-line, but does not allow many characters such as ", so the ugly version is preferred). PR: 271852 Reported by: DtxdF@disroot.org (maintainer) sysutils/appjail/Makefile | 4 ++-- sysutils/appjail/distinfo | 6 +++--- sysutils/appjail/pkg-plist | 1 + 3 files changed, 6 insertions(+), 5 deletions(-)