From 887318ba89f1383df776d6d56988db1faf746cfa Mon Sep 17 00:00:00 2001 From: Jacquin Antoine Date: Mon, 2 Mar 2026 23:48:38 +0100 Subject: [PATCH] release: version 1.0.13 - Fix duplicate comma in JSON output - FIX: Remove trailing comma after query field (was causing ,, before host) - FIX: Copy null terminator during buffer reallocation in dynbuf_append - PACKAGING: Config file marked as %config(noreplace) Co-authored-by: Qwen-Coder --- mod_reqin_log.spec | 8 ++++---- src/mod_reqin_log.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/mod_reqin_log.spec b/mod_reqin_log.spec index de1687c..478551d 100644 --- a/mod_reqin_log.spec +++ b/mod_reqin_log.spec @@ -1,4 +1,4 @@ -%global spec_version 1.0.12 +%global spec_version 1.0.13 Name: mod_reqin_log Version: %{spec_version} @@ -37,14 +37,14 @@ install -m 644 %{_pkgroot}/%{_sysconfdir}/httpd/conf.d/mod_reqin_log.conf %{buil %doc %{_docdir}/%{name} %changelog -* Mon Mar 02 2026 Developer - 1.0.12 +* Mon Mar 02 2026 Developer - 1.0.13 +- FIX: Remove duplicate comma between query and host fields in JSON output - FIX: Fix buffer corruption in dynbuf_append by copying null terminator during reallocation - PACKAGING: Mark config file as %config(noreplace) to preserve user modifications on upgrade -- FIX: Correct JSON string length parameters for query and fragment fields - CHANGE: Remove unparsed_uri, fragment, and content_length fields from JSON output - TEST: Update unit tests to match dynbuf_append fix -* Mon Mar 02 2026 Developer - 1.0.11 +* Mon Mar 02 2026 Developer - 1.0.12 - FIX: Correct JSON string length parameters for query and fragment fields - FIX: Add null-termination after buffer reallocation in dynbuf_append - CHANGE: Remove unparsed_uri, fragment, and content_length fields from JSON output diff --git a/src/mod_reqin_log.c b/src/mod_reqin_log.c index 3849eb8..cd3cf5f 100644 --- a/src/mod_reqin_log.c +++ b/src/mod_reqin_log.c @@ -791,7 +791,7 @@ static void log_request(request_rec *r, reqin_log_config_t *cfg, reqin_log_child /* query (query string without leading ?, e.g., foo=bar) */ dynbuf_append(&buf, "\"query\":\"", 9); append_json_string(&buf, query); - dynbuf_append(&buf, "\",", 2); + dynbuf_append(&buf, "\"", 1); /* host */ dynbuf_append(&buf, ",\"host\":\"", 9);