patchlwIP - A Lightweight TCP/IP stack - Patches: patch #9429, mdns: added support for searching...

 
 

You are not allowed to post comments on this tracker with your current authentication level.

patch #9429: mdns: added support for searching services

Submitter:  David GIRAULT <dgirault>
Submitted:  Mon 07 Aug 2017 01:30:45 PM UTC
   
 
Category:  DNS Priority:  5 - Normal
Status:  Done Privacy:  Public
Assigned to:  goldsimon Open/Closed:  Closed
Planned Release:  None

Jump to the original submission

Fri 08 Feb 2019 06:37:40 AM UTC, comment #8: 

How does this compare to my work on DNS look-ups of other RRs in task #15104?

Craig McQueen <cmcqueen1975>
Wed 30 Jan 2019 07:16:00 PM UTC, comment #7: 

Pushed, thanks for the patch.

Simon Goldschmidt <goldsimon>
Group administrator
Tue 29 Jan 2019 03:42:41 PM UTC, comment #6: 

Just got time to update my patchset on our repository to updated upstream.

My last comment #5 was not yet addressed. Sorry, not enough time for that.

(file #46122)

David GIRAULT <dgirault>
Wed 10 Jan 2018 02:57:07 PM UTC, comment #5: 

Erik, after reading the rfc, I saw the point. My mdns_search() need to use another udp_pcb to send the request. A new temporary udp_pcb with the same recv handle should do the job.

David GIRAULT <dgirault>
Tue 09 Jan 2018 01:17:56 PM UTC, comment #4: 

This patch allow search for services like `_workstation._tcp.local` using PTR type query.

DNS module only support querying IP for a specified hostname (A or AAAA queries).

David GIRAULT <dgirault>
Mon 08 Jan 2018 09:02:43 PM UTC, comment #3: 

The DNS module already supports sending queries for the .local. domain to the MDNS multicast address. What does this add?

It sends queries with source port 5353 even though it is not a full MDNS capable querier. This is not allowed by the RFC:
(rfc6762 section 5.1)
.... these queries MUST NOT be sent using UDP source port 5353, since using UDP source port 5353 signals the presence of a fully compliant Multicast DNS querier, as described below.

Erik Ekman <yarrick>
Group Member
Mon 08 Jan 2018 01:40:47 PM UTC, comment #2: 

Since some mDNS work are ongoing, I thinks this path should be considered.
Regards,
David

David GIRAULT <dgirault>
Mon 07 Aug 2017 01:53:02 PM UTC, comment #1: 

Second patch, `0002-mdns-early-uncompress-domain-name-since-cannot-be-do.patch` is the only addition to original #9339 patch.

David GIRAULT <dgirault>
Mon 07 Aug 2017 01:30:45 PM UTC, original submission:  


This new patchset was previously #9339, but this new one include only the search part (responder part already merged).

All comments previously posted on #9339 were already addressed.


David GIRAULT <dgirault>

 

(Note: upload size limit is set to 16384 kB, after insertion of the required escape characters.)

 

Depends on the following items: None found

Items that depend on this one: None found

 

Carbon-Copy List
  • -email is unavailable- added by cmcqueen1975 (Posted a comment)
  • -email is unavailable- added by goldsimon (Posted a comment)
  • -email is unavailable- added by yarrick (Posted a comment)
  • -email is unavailable- added by dgirault (Submitted the item)
  •  

    There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.

     

    Follow 6 latest changes.

    Date Changed by Updated Field Previous Value => Replaced by
    2019-01-30 goldsimon StatusNone Done
        Assigned toNone goldsimon
        Open/ClosedOpen Closed
    2019-01-29 dgirault Attached File- Added 0001-mdns-added-support-for-searching-services.patch, #46122
    2017-08-07 dgirault Attached File- Added 0001-mdns-added-support-for-searching-services.patch, #41441
        Attached File- Added 0002-mdns-early-uncompress-domain-name-since-cannot-be-do.patch, #41442

    Back to the top

    Powered by Savane 3.13-758e.
    Corresponding source code