Mercurial > hg > webserver / file diff

--- a/src/server/daemon/httprequest.c	Tue Nov 11 11:03:50 2025 +0100
+++ b/src/server/daemon/httprequest.c	Tue Nov 11 16:53:23 2025 +0100
@@ -337,6 +337,16 @@
                 absPath.length,
                 rq->rq.reqpb);
     } else {
+        // util_uri_unescape_strict can modify absPath.ptr, but
+        // we want to log the original uri. However we also don't want to
+        // create an unnecessary copy. Therefore we restore the original
+        // uri here by calling util_canonicalize_uri again
+        absPath.ptr = util_canonicalize_uri(
+            pool,
+            absPath.ptr,
+            absPath.length,
+            (int*)&absPath.length);
+        
         log_ereport(
                 LOG_FAILURE,
                 "uri unescape failed: {%.*s}",
@@ -345,19 +355,6 @@
         request->status = 400;
         //pblock_kvinsert(pb_key_uri, "/", 1, rq->rq.reqpb);
         
-        // TODO: remove this debug code
-        char tmp_file_path[128];
-        snprintf(tmp_file_path, 128, "logs/req_uri_fail_%lld", (long long int)time(NULL));
-        log_ereport(LOG_FAILURE, "uri unescape req file: %s\n", tmp_file_path);
-        FILE *f = fopen(tmp_file_path, "w");
-        if(f) {
-            fwrite(request->netbuf->inbuf, 1, request->netbuf->pos, f);
-            fclose(f);
-        } else {
-            log_ereport(LOG_FAILURE, "Cannot write req uri fail file\n");
-        }
-        
-        
         return 1;
     }