[joeig]

> Sometimes users will still get errors from PowerDNS directly, but I added some logging of all the errors that users see, so hopefully I can review them and add extra translations if there are other common errors that come up.

I noticed that you are using our Go module to access the API. It is wonderful to see our work helping others build great software, especially for education. Thank you for that.

Please note that the upstream API sometimes changes slightly between minor releases. For example, prior to v4.9, the error response for a non-existent server was "Not Found". Starting with v4.9, it changed to "Method Not Allowed".

Unfortunately, error responses aren't always part of the API specification. I'm thinking about adding the most common cases to the module anyway.

[remram]

You just said that the message for "non-existent server" is spelled "method not allowed"? Did I read this right?

[justinclift]

Seems like a super confusing response message for a server not being found. :(

[grantla]

Care to expand on the "non-existent server" part? This is probably an oversight (missing tests) in the API impl.