Skip to content
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

Issue 6739-Added custom DNS server for hostname resolution(Needs fixing) #6815

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

smenon02
Copy link

  • Description

Creates a DNSServer addon for hostname resolution. Python implementation includes an async getaddrinfo method that opens a UDP socket to send DNS queries to using hostname to resolve. Running into issues while testing as communication between UDP server and DNS client is timing out. Any assistance of this would be greatly appreciated :)
UDP server was created to test hostname resolution. DNS client is not properly talking to it, so once communication between the two is fixed, test cases can be generated.

Checklist

  • I have updated tests where applicable.
  • I have added an entry to the CHANGELOG.

smenon02 and others added 3 commits April 21, 2024 13:01
…er that opens UDP socket to send DNS queries for hostname resolution. Running into troubles getting UDP socket and DNS server to communicate, leading to timeout issues. Some help with this would be greatly appreciated :)
Copy link
Member

@mhils mhils left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR. The async_getaddrinfo method is neither async nor does it implement a getaddrinfo-equivalent. So this should be fixed before we get to any tests.

@mhils mhils marked this pull request as draft April 22, 2024 13:45
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.

None yet

2 participants