Stop assuming signed chars in DUMP_STRING(); that results in inconsistent output in platforms like ARM64 #34

Author: carlosabalde

docker-compose.yml

@@ -19,6 +19,7 @@ name: libvmod-cfg-8_0
 services:
   dev:
     hostname: dev
+    # platform: linux/arm64/v8
     build:
       context: .
     privileged: true

src/variables.c

@@ -129,7 +129,7 @@ static const char *json_hex_chars = "0123456789abcdef";
     do { \
         DUMP_CHAR('"'); \
         for (int i = 0; value[i]; i++) { \
-            if (value[i] > 31 && value[i] != '\"' && value[i] != '\\') { \
+            if (value[i] > 31 && value[i] < 127 && value[i] != '\"' && value[i] != '\\') { \
                 DUMP_CHAR(value[i]); \
             } else { \
                 DUMP_CHAR('\\'); \