Chris,
Sorry, but I haven’t tried your solution. I wanted something that would work from the highest level, so I created the function function netconn_gethostCname(). Whereas our solution only changed dns.c., my changes are spread across several files, and are conditionalized with CNAME_HACK. If you call netconn_gethostCname( myHostname, myHostCname), Where myHostName is the URL for which you want the CNAME, and myHostCanem is where you want the result stored, it should work. For example:
hostCNameTask Looking up CName for '16662.lb.invue-am.com'
hostCNameTask CName 'dev-ir4-sso.invue-am.com'
Jeff Thompson | Senior Electrical Engineer-Firmware
+1 704 752 6513 x1394
www.invue.com
From: Christopher Nocker <nocker02@gmail.com>
Sent: Wednesday, July 15, 2020 09:44
To: Thompson, Jeff <JeffThompson@invue.com>
Cc: Simon Goldschmidt <goldsimon@gmx.de>; Gao Qingshui <gaoqingshui202@163.com>; Sylvain Rochet <gradator@gradator.net>; lwip-devel@nongnu.org; lwip-members@nongnu.org
Subject: Re: [bug #56280] lwIP DNS need to support CNAME Resource Record
Hi Jeff,
Our implementations are slightly different in what they are trying to achieve. I was trying to iterate over the response from the DNS server to see if an A record was present (i.e. when the DNS server would have done the recursion and all of the CNAMEs for the query would be in the response). Your implementation is explicitly looking for the CNAME response to a query.
I have tried your implementation and queried a name which I know has a couple of CNAMEs in the response but my application was never able to resolve it. Maybe I'm using it wrong? I'm guessing that it is working for you?
Did you try my implementation? Does my implementation suit your needs?
Regards,
Chris
On Tue, Jun 30, 2020 at 8:21 PM Thompson, Jeff <JeffThompson@invue.com> wrote:
Attached is the solution I came up with. All my changes are conditionalized the macro CNAME_HACK. The entry point is the function netconn_gethostCname().
Jeff Thompson | Senior Electrical Engineer-Firmware
+1 704 752 6513 x1394
www.invue.com
-----Original Message-----
From: Chris Nocker <INVALID.NOREPLY@gnu.org>
Sent: Monday, June 29, 2020 23:45
To: Simon Goldschmidt <goldsimon@gmx.de>; Gao Qingshui <gaoqingshui202@163.com>; Thompson, Jeff <JeffThompson@invue.com>; Chris Nocker <nocker02@gmail.com>; Sylvain Rochet <gradator@gradator.net>; lwip-devel@nongnu.org; lwip-members@nongnu.org
Subject: [bug #56280] lwIP DNS need to support CNAME Resource Record
Additional Item Attachment, bug #56280 (project lwip):
File name: 0001-DNS-Allowed-processing-of-a-DNS-response-that-contai.patch
Size:1 KB
<https://savannah.nongnu.org/file/0001-DNS-Allowed-processing-of-a-DNS-response-that-contai.patch?file_id=49395>
_______________________________________________________
Reply to this item at:
<https://savannah.nongnu.org/bugs/?56280>
_______________________________________________
Message sent via Savannah
https://savannah.nongnu.org/
