-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Update DNS Unit/Integration Tests #1010
Comments
Thanks Kevin. Please return the tests ASAP.
|
Hey @dnkolegov I was wondering if you and your team could maybe have a look into this? |
Blocked on #1253 |
The unit and integration tests for the DNS extension are currently disabled since they are still using the old DNS API. They need to be updated to make use of the new API.
I plan to return to this issue in the future. At the moment, there are bigger fish to fry - namely DNS tunneling - and have to put this on the back burner for the time being. If anyone feels they can handle this in the meantime, they are certainly free to do so.
The text was updated successfully, but these errors were encountered: