daemon/http: ensure HTTP headers are freed

This should solve a memory leak added in !1165 (merged).

---

Still needs more testing and perhaps a test deployment before merging.

Test deployment revelead additional issue which is now fixed

changed milestone to %5.4.0

The test case I had seems fixed by this.

resolved all threads

added 63 commits

• 273b22cd - log: add log levels
• af760759 - log: remove kr_verbose_set
• 1701d9ec - log: remove kr_verbose_status
• 1ccef671 - log: add syslog target, systemd
• 340de441 - log: lua interface
• a242be26 - log: split to groups
• 947188a6 - log: fix journal metadata
• fb31ed28 - log: groups lua interface
• 8042acfa - fixup! log: remove kr_verbose_set
• 51322fae - fixup! log: lua interface
• 92116635 - fixup! log: split to groups
• 4c4e7740 - fixup! log: remove kr_verbose_set
• a0077e53 - fixup! log: lua interface
• 0a2ba6c3 - fixup! log: groups lua interface
• f30da41f - fixup! log: add syslog target, systemd
• d916574c - fixup! log: split to groups
• f4100a61 - fixup! log: groups lua interface
• fb590841 - fixup! log: add syslog target, systemd
• 8c20365a - log: lua: add list_log_group
• bce97844 - log: remove NOVERBOSELOG option
• e8298a4b - log: add level prefix to each message
• 6545282f - log: remove hard-coded prefix from log messages
• f89b6b71 - log: shortening LOG_GRP_<SOMETHING> to <SOMETHING>
• 405479cf - log: fix CI tests
• da606f3e - log: lua logging
• d3421eaf - log: fix CI tests
• f1c7d357 - log: use kr_log_debug instead kr_log_verbose
• 857832b4 - log: fix kr_log_fmt SIGSEGV crash
• cb0b4975 - fixup! log: fix CI tests
• ae69af9d - daemon/main: log interactive message properly
• 274ac0f5 - lib: make NOTICE the default log level
• b6fefb32 - logging: ensure get/set_log_level behaves consistently
• 882b75b9 - logging: add get_log_group() with better API
• ca8d527c - logging: print group name padded to 6 characters
• c22a9a64 - logging: add LOG_GRP_DEVEL
• 2900c9de - logging: fix get_log_group()
• 3b4bd5f5 - logging: add missing LOG_GRP_PLAN
• 4952066d - logging: add test for log groups
• 862002ae - logging: rename *_log_groups() function to plural
• d3fd8305 - logging: use dedicated DOTAUTH log for experimental auth DoT
• c97ae9f9 - logging: nitpicks
• 8357f14c - logging: use LOG_GRP_HTTP for http lua module
• 63b1167c - logging: fix GNUTLS logging
• b302ecd7 - fixup! logging: use dedicated DOTAUTH log for experimental auth DoT
• 7f67b491 - fixup! logging: use LOG_GRP_HTTP for http lua module
• 55230825 - lib/log: typing improvements
• ae0b685e - QRVERBOSE: fix ignoring the log groups
• ae559989 - rework logging from control sockets (+simplify)
• a8b7e6e9 - lib/log: make trace-logging not prevent normal logging
• a8d3468d - fixup! rework logging from control sockets (+simplify)
• 822867f4 - lib/log: add kr_log_is_debug*()
• 5e2be069 - validate: add kr_rrset_validation_ctx_t::log_qry
• c781cef8 - treewide: replace VERBOSE_STATUS, WITH_VERBOSE, etc.
• d17790fe - lib/utils: drop a pair of unused macros
• 163ab9c1 - fixup! treewide: replace VERBOSE_STATUS, WITH_VERBOSE, etc.
• b7793003 - fixup! log: lua logging
• 42f6ac26 - daemon/http: refactor freeing headers into function
• 69396487 - daemon/http: ensure headers are freed
• 4415667d - daemon/http: ensure missing variable in URI is treated as error
• b26b48e8 - daemon/http: refactor submitting to wirebuffer
• 438c804b - daemon/worker: ensure HTTP headers are freed on ignored queries
• 9a4f23e3 - daemon/http: add helper assert to detect potential memleak
• 5e133221 - daemon/http: add more info to debug messagess

Compare with previous version

added 1 commit

• 6155ada2 - daemon/http: add more info to debug messagess

Compare with previous version

added 1 commit

• 78fa0e9c - daemon/http: fix crash on large buffers and document logic issues

Compare with previous version

added 99 commits

• 78fa0e9c...d8a07396 - 91 commits from branch master
• 02c12da2 - daemon/http: refactor freeing headers into function
• 8e03541f - daemon/http: ensure headers are freed
• 96fa92b3 - daemon/http: ensure missing variable in URI is treated as error
• fc6f73e6 - daemon/http: refactor submitting to wirebuffer
• 82be9010 - daemon/worker: ensure HTTP headers are freed on ignored queries
• 298003f7 - daemon/http: add helper assert to detect potential memleak
• 44174428 - daemon/http: add more info to debug messagess
• 39372ab2 - daemon/http: fix crash on large buffers and document logic issues

Compare with previous version

marked this merge request as ready

changed the description

approved this merge request

mentioned in commit d2fcd60a

merged