commit 03840a380d80ba1aab14fd7b83fb90c2e1db2bd5
author Sergey Kandaurov <pluknet@nginx.com> Mon Jun 28 17:13:45 2021 +0300
committer Sergey Kandaurov <pluknet@nginx.com> Mon Jun 28 17:13:45 2021 +0300
tree 5bd611c987bab63561ba3689344f079e3b24b85d
parent 33821c62facc10ba344b3c5b1990ca955e450c18

Tests: adjusted access_log escaping tests.

Since HTTP arguments are soon to require stricter escaping,
control characters and spaces in testing data are now logged
through the $uri variable.

access_log.t

diff --git a/access_log.t b/access_log.t
index 8443a6a..baf1b55 100644
--- a/access_log.t
+++ b/access_log.t
@@ -40,9 +40,9 @@
     log_format addr "$remote_addr:$remote_port:$server_addr:$server_port";
     log_format binary $binary_remote_addr;
 
-    log_format default  escape=default  $arg_a$arg_b$arg_c;
-    log_format none     escape=none     $arg_a$arg_b$arg_c;
-    log_format json     escape=json     $arg_a$arg_b$arg_c;
+    log_format default  escape=default  $uri$arg_b$arg_c;
+    log_format none     escape=none     $uri$arg_b$arg_c;
+    log_format json     escape=json     $uri$arg_b$arg_c;
 
     server {
         listen       127.0.0.1:8080;
@@ -169,7 +169,8 @@
 
 http_get('/binary');
 
-http_get('/escape?a="1 \\ ' . pack("n", 0x1b1c) . ' "&c=2');
+# /escape/"1 %1B%1C "?c=2
+http_get('/escape/%221%20%1B%1C%20%22?c=2');
 
 http_get('/cache?logname=lru');
 http_get('/cache?logname=lru');
@@ -271,11 +272,11 @@
 # characters escaping
 
 is($t->read_file('test.log'),
-	'\x221 \x5C \x1B\x1C \x22-2' . "\n", 'escape - default');
+	'/escape/\x221 \x1B\x1C \x22-2' . "\n", 'escape - default');
 is($t->read_file('none.log'),
-	'"1 \\ ' . pack("n", 0x1b1c) . " \"2\n", 'escape - none');
+	"/escape/\"1 \x1B\x1C \"2\n", 'escape - none');
 is($t->read_file('json.log'),
-	'\"1 \\\\ \u001B\u001C \"2' . "\n", 'escape - json');
+	'/escape/\"1 \u001B\u001C \"2' . "\n", 'escape - json');
 
 SKIP: {
 skip 'win32', 4 if $^O eq 'MSWin32';