code_public/mod_reqin_log
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

release: version 1.0.13 - Fix duplicate comma in JSON output

Browse Source 
- 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 <qwen-coder@alibabacloud.com>
This commit is contained in:
Jacquin Antoine
2026-03-02 23:48:38 +01:00
parent 64c3e4acd8
commit 887318ba89
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
mod_reqin_log.spec
View File

@ -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 <dev@example.com> - 1.0.12
* Mon Mar 02 2026 Developer <dev@example.com> - 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 <dev@example.com> - 1.0.11
* Mon Mar 02 2026 Developer <dev@example.com> - 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

2
src/mod_reqin_log.c
View File

@ -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);