Skip to content
Snippets Groups Projects

daemon/proxyv2: fix client TCP/TLS + merge state structs

Merged Oto Šťáva requested to merge proxyv2-tcp-fix into master
All threads resolved!

Firstly, this adds a missing assignment of comm->proxy, which would cause the resolver not to know that a client asked via TCP/TLS, if the proxy also asked via TCP/TLS.

Secondly, it merges struct pl_proxyv2_sess_data and struct pl_proxyv2_iter_data into a single struct pl_proxyv2_state, since they were practically the same. And it adds a bit of documentation to the struct.

Merge request reports

Pipeline #128616 waiting for manual action

Pipeline: Knot Resolver

#128617

    Pipeline waiting for manual action for adc4c44e on proxyv2-tcp-fix

    Deployed to docs-develop/p‎roxyv2-tcp-fix‎ 8 months ago
    Approved by

    Merged by Oto ŠťávaOto Šťáva 8 months ago (Aug 7, 2024 8:59am UTC)

    Merge details

    • Changes merged into master with dc8e2da6.
    • Deleted the source branch.

    Pipeline #128618 failed

    Pipeline failed for dc8e2da6 on master

    Deployed to docs-deve‎lop/master‎ 8 months ago

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
    Please register or sign in to reply
    Loading