lwIP - A Lightweight TCP/IP stack - Patches: patch #9721, DNS: Allow a DNS look-up with a...
You are not allowed to post comments on this tracker with your current authentication level.
patch #9721: DNS: Allow a DNS look-up with a trailing dot in the name
Submitter: | Craig McQueen <cmcqueen1975> | ||
Submitted: | Tue 13 Nov 2018 11:37:30 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
Thu 22 Nov 2018 12:15:25 PM UTC, comment #7: |
Simon Goldschmidt <goldsimon> |
Thu 22 Nov 2018 01:14:44 AM UTC, comment #6: I found an additional change that I should make. I'm adding an updated patch.
|
Craig McQueen <cmcqueen1975> |
Fri 16 Nov 2018 02:18:11 AM UTC, comment #5: I'm providing an updated patch. Is this more suitable? |
Craig McQueen <cmcqueen1975> |
Thu 15 Nov 2018 10:30:07 AM UTC, comment #4:
|
Simon Goldschmidt <goldsimon> |
Wed 14 Nov 2018 11:57:07 PM UTC, comment #3: That makes sense.
|
Craig McQueen <cmcqueen1975> |
Wed 14 Nov 2018 07:45:27 PM UTC, comment #2: Your patch works, but I would prefer to strip the trailing dot a lot earlier. The goal would be that the trailing dot is only used as an indicator to not try the search list (once we implement that) and that a cached value matches for both queries (with or without trailing dot).
|
Simon Goldschmidt <goldsimon> |
Wed 14 Nov 2018 01:45:49 AM UTC, comment #1: See task #14129 regarding DNS domain search list. |
Craig McQueen <cmcqueen1975> |
Tue 13 Nov 2018 11:37:30 PM UTC, original submission:
This is not a well-known behaviour of DNS names, but a trailing dot in a domain name is technically correct to make it a fully-qualified domain name. I.e. it won't search by appending any DNS search domains.
|
Craig McQueen <cmcqueen1975> |
Depends on the following items: None found
Items that depend on this one: None found
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 |
---|---|---|---|---|---|
2018-11-22 | goldsimon | Status | None | Done | |
Assigned to | None | goldsimon | |||
Open/Closed | Open | Closed | |||
2018-11-22 | cmcqueen1975 | Attached File | - | Added 0001-DNS-Allow-a-DNS-look-up-with-a-trailing-dot-in-the-n.patch, #45509 | |
2018-11-16 | cmcqueen1975 | Attached File | - | Added 0001-DNS-Allow-a-DNS-look-up-with-a-trailing-dot-in-the-n.patch, #45442 | |
2018-11-13 | cmcqueen1975 | Attached File | - | Added 0001-DNS-Allow-a-DNS-look-up-with-a-trailing-dot-in-the-n.patch, #45420 |
Powered by Savane 3.13-758e.
Corresponding source code
Yes. Pushed, thanks for the patch.