Re: [dev] suckless dns over http client

From: Martin Tournoij <martin_AT_arp242.net>
Date: Tue, 03 Apr 2018 21:17:13 +0100

On Tue, Apr 3, 2018, at 21:12, harry666t wrote:
> My take - if you don't mind Go...
> https://github.com/rollcat/gdoh
> No forking, no dependencies outside of stdlib, async
> queries/responses, allows using multiple providers, 78 loc.

There is a small bug on line 34: if the statuscode isn't 200 then the
response body is never closed. You should move the "defer
r.Body.Close()" to above the "if r.StatusCode != 200 {" check.

Probably not a huge deal since this is a command line client, but in
long-running servers this will cause file descriptor leaks.
Received on Tue Apr 03 2018 - 22:17:13 CEST

This archive was generated by hypermail 2.3.0 : Tue Apr 03 2018 - 22:24:30 CEST