Opened 8 years ago

Closed 6 years ago

#2239 closed task (wontfix)

change UDP/TCP/SyncUDPServer so they don't pass query message object

Reported by: jinmei Owned by:
Priority: medium Milestone: DNS Outstanding Tasks
Component: b10-auth Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket:
Estimated Difficulty: 6 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description (last modified by jinmei)

In #2160, we changed SyncUDPServer so it doesn't clear the query_
Message object. In fact, it could even remove that member variable
if it wasn't requested by the API contract. Unfortunately we cannot
simply remove them without making non trivial changes to UDPServer
and TCPServer classes and the b10-resolver implementation.

IMO, it's better to make the xxx server classes independent from
Message management. This ticket is a placeholder to do this task.
Specifically:

  • remove the "query" parameter from the callback functions
  • remove the "query" member variables from XXX server classes
  • update b10-auth so it maintains Message internally (now that it's not given via the callbacks). This should be easy.
  • Same for b10-resolver. This may be a bit trickier due to the recursive-resume operations.

Subtickets

Change History (4)

comment:1 Changed 8 years ago by jinmei

  • Description modified (diff)

comment:2 Changed 7 years ago by shane

  • Milestone New Tasks deleted

comment:3 Changed 6 years ago by stephen

  • Milestone set to DNS Outstanding Tasks

comment:4 Changed 6 years ago by tomek

  • Resolution set to wontfix
  • Status changed from new to closed

DNS and BIND10 framework is outside of scope for Kea project.
The corresponding code has been removed from Kea git repository.
If you want to follow up on DNS or former BIND10 issues, see
http://bundy-dns.de project.

Closing ticket.

Note: See TracTickets for help on using tickets.