Skip to content

Dev ro schmi #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from
Closed

Dev ro schmi #6

wants to merge 7 commits into from

Conversation

RoSchmi
Copy link

@RoSchmi RoSchmi commented Jun 10, 2021

When working with the NativeEthernet library which uses FNET as dependency I noticed that certain urls were not resolved.
I opened an issue on butok / FNET but we didn't succeed to achieve a solution.
butok#7
Others like @egonbeermat and @frederic01 (on the teensy forum https://forum.pjrc.com/) had similar problems that urls using aliasis were not resolved.
https://forum.pjrc.com/threads/67252-Teensyduino-1-54-Beta-9?p=280976#post280976
defragster at the pjrc forum proposed to make a pull request against https://github.com/vjmuzik/FNET
My changes solved the problems for me and @egonbeermat but I propose that the changes are reviewed so that they will not have unforeseen sideeffects.

@vjmuzik
Copy link
Owner

vjmuzik commented Aug 14, 2021

Try this last commit and see if it resolves this issue for you, I'm finally getting around to doing some improvements and I discovered the UdpNtpClient example from NativeEthernet wasn't obtaining an IP. Of course Wireshark shows that it got something and I remembered this issue so I added some handling for a CNAME record based on what I think should happen with it. Now the example is working so this should presumably fix your issue too.

@RoSchmiSec
Copy link

Thanks, I'll try it in the next days and will give feedback.

@RoSchmiSec
Copy link

I can confirm that with your last commit (be3a67e) and arduinoteensy 1.154 dns resolving of CNAME records works for my application. Thank you.

@vjmuzik
Copy link
Owner

vjmuzik commented Aug 20, 2021

I'll close this for now, but reopen it if you run into any other urls giving you an issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants