-
Notifications
You must be signed in to change notification settings - Fork 497
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
Start decoding EDE #969
Labels
Comments
I would expect something like list of EDEs, where each item is [enum-value, extra-text]. It probably makes sense to tack it onto Exceptions thrown from resolver, but for non-error cases I think DNS message might get a helper property to extract this from EDNS. |
Yes that makes sense.
G
…On Thu, 27 July 2023, 17:08 Petr Špaček, ***@***.***> wrote:
I would expect something like list of EDEs, where each item is
[enum-value, extra-text]. It probably makes sense to tack it onto
Exceptions thrown from resolver, but for non-error cases I think DNS
message might get a helper property to extract this from EDNS.
—
Reply to this email directly, view it on GitHub
<#969 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ABORQ7COHX6QYXW2V2CSEH3XSIHX7ANCNFSM6AAAAAA2ZOFCIY>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
rthalley
added a commit
that referenced
this issue
Feb 19, 2024
rthalley
added a commit
that referenced
this issue
Feb 20, 2024
rthalley
added a commit
that referenced
this issue
Feb 20, 2024
basic support added |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
Motivation
The beginnings of extended error codes are visible in the source.
Cloudflare public DNS now returns them and "dig" interprets them into comments.
I'd like to be able to "see" these as decomposed high level constructs inside dnspython.
Describe the solution you'd like.
Could adorn the Exception classes? or, they could be some other datastruct. I think their main use is as adjunct information to error/fail/nxdomain/timeout type responses which is why I think it might be populating the exception classes.
The text was updated successfully, but these errors were encountered: