]> git.ipfire.org Git - thirdparty/knot-dns.git/commitdiff
Revert "mod-dnstap: Restore the original query QNAME case"
authorRobert Edmonds <edmonds@users.noreply.github.com>
Wed, 5 Jan 2022 22:18:23 +0000 (17:18 -0500)
committerDaniel Salzman <daniel.salzman@nic.cz>
Mon, 10 Jan 2022 07:49:29 +0000 (08:49 +0100)
This reverts commit 4a3eb76be83b99b100a8289253205d6e7559821f.

src/knot/include/module.h
src/knot/modules/dnstap/dnstap.c
src/knot/nameserver/query_module.c

index 47ccb85e7575a4fd9f4b9b1f6b558d5b3c0bd34d..e2738e66af8ac90b7bade946e9698e44726cc493 100644 (file)
@@ -465,15 +465,6 @@ const knot_dname_t *knotd_qdata_zone_name(knotd_qdata_t *qdata);
  */
 knot_rrset_t knotd_qdata_zone_apex_rrset(knotd_qdata_t *qdata, uint16_t type);
 
-/*!
- * Gets the original QNAME of the query, before downcasing.
- *
- * \param[in] qdata  Query data.
- *
- * \return Original QNAME or NULL if error.
- */
-const knot_dname_t *knotd_qdata_orig_qname(knotd_qdata_t *qdata);
-
 /*! General query processing states. */
 typedef enum {
        KNOTD_STATE_NOOP  = 0, /*!< No response. */
index 60a4eb6cacaaaf9c9e54e59ee04cfd89de2530a8..eef68bd3d35e7be0947eb9cebddbd66edaef9e08 100644 (file)
@@ -61,30 +61,6 @@ typedef struct {
        bool with_queries;
 } dnstap_ctx_t;
 
-static void msg_query_qname_restore(Dnstap__Message *msg, knotd_qdata_t *qdata)
-{
-       if (msg->query_message.data == NULL) {
-               return;
-       }
-
-       const knot_dname_t *orig_qname = knotd_qdata_orig_qname(qdata);
-       if (orig_qname == NULL) {
-               return;
-       }
-
-       memcpy(msg->query_message.data + KNOT_WIRE_HEADER_SIZE,
-              orig_qname, qdata->query->qname_size);
-}
-
-static void msg_query_qname_case_lower(Dnstap__Message *msg)
-{
-       if (msg->query_message.data == NULL) {
-               return;
-       }
-
-       knot_dname_to_lower(msg->query_message.data + KNOT_WIRE_HEADER_SIZE);
-}
-
 static knotd_state_t log_message(knotd_state_t state, const knot_pkt_t *pkt,
                                  knotd_qdata_t *qdata, knotd_mod_t *mod)
 {
@@ -156,9 +132,7 @@ static knotd_state_t log_message(knotd_state_t state, const knot_pkt_t *pkt,
        /* Pack the message. */
        uint8_t *frame = NULL;
        size_t size = 0;
-       msg_query_qname_restore(&msg, qdata);
        dt_pack(&dnstap, &frame, &size);
-       msg_query_qname_case_lower(&msg);
        if (frame == NULL) {
                return state;
        }
index 976b0576d22869108d852854b664ae333c7e8709..0effefe524004a462340c2aabdcd742a9fe47e9d 100644 (file)
@@ -681,16 +681,6 @@ knot_rrset_t knotd_qdata_zone_apex_rrset(knotd_qdata_t *qdata, uint16_t type)
        return node_rrset(qdata->extra->contents->apex, type);
 }
 
-_public_
-const knot_dname_t *knotd_qdata_orig_qname(knotd_qdata_t *qdata)
-{
-       if (qdata == NULL) {
-               return NULL;
-       }
-
-       return qdata->extra->orig_qname;
-}
-
 _public_
 int knotd_mod_dnssec_init(knotd_mod_t *mod)
 {