diff mbox series

[2/4] jshn.sh: Drop extraneous space when appending

Message ID mailman.44919.1746395808.1866309.openwrt-devel@lists.openwrt.org
State New
Headers show
Series [1/4] jshn.sh: Send warnings to stderr | expand

Commit Message

Philip Prindeville May 4, 2025, 9:56 p.m. UTC 
The sender domain has a DMARC Reject/Quarantine policy which disallows
sending mailing list messages using the original "From" header.

To mitigate this problem, the original message has been wrapped
automatically by the mailing list software.
From: Philip Prindeville <philipp@redfish-solutions.com>

Don't unnecessarily add a space before the first value when
appending to a list.

Signed-off-by: Felix Fietkau <nbd@nbd.name>
Signed-off-by: Philip Prindeville <philipp@redfish-solutions.com>
---
 sh/jshn.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff mbox series

Patch

diff --git a/sh/jshn.sh b/sh/jshn.sh
index 3f2f679187db96d77ccf27b15f5fc47cb8ee9796..3657642fd7673eeb417b0148af3e0b7c1e5c0825 100644
--- a/sh/jshn.sh
+++ b/sh/jshn.sh
@@ -23,7 +23,7 @@  __jshn_raw_append() {
 _jshn_append() {
 	# var=$1
 	local _a_value="$2"
-	eval "${JSON_PREFIX}$1=\"\${${JSON_PREFIX}$1} \$_a_value\""
+	eval "${JSON_PREFIX}$1=\"\${${JSON_PREFIX}$1}\${${JSON_PREFIX}$1:+ }\$_a_value\""
 }
 
 _get_var() {