Re: NEW: net/doh

2019-09-14 Thread Stuart Henderson
On 2019/09/14 20:34, Gleydson Soares wrote:
> 
> 
> > On 14 Sep 2019, at 20:28, Stuart Henderson  wrote:
> > 
> >> On 2019/09/13 12:57, Gleydson Soares wrote:
> >> i have fixed the segfault by adding a patch and also i put the pkg-readme.
> >> pull request done, upstream aware.
> >> 
> >> this one is OK gsoares@
> > 
> > I don't like hand-copying README.md to pkg/README at all.
> > 
> > README.md could be installed to PREFIX/share/doc/doh but it doesn't seem
> > very useful as it's right there on HOMEPAGE anyway, plus it's such a
> > simple utility ...
> > 
> 
> Either as it turns out or removing readme from pkg/ OK gsoares@
> 
> Looks like upstream merged a fix addressing the segfault, the PR was closed 
> such that GH_COMMIT should be updated + get rid of patch-doh-c.
> 
> BR,
> Gleydson.
> 
> 
> 

Thanks.



Re: NEW: net/doh

2019-09-14 Thread Gleydson Soares



> On 14 Sep 2019, at 20:28, Stuart Henderson  wrote:
> 
>> On 2019/09/13 12:57, Gleydson Soares wrote:
>> i have fixed the segfault by adding a patch and also i put the pkg-readme.
>> pull request done, upstream aware.
>> 
>> this one is OK gsoares@
> 
> I don't like hand-copying README.md to pkg/README at all.
> 
> README.md could be installed to PREFIX/share/doc/doh but it doesn't seem
> very useful as it's right there on HOMEPAGE anyway, plus it's such a
> simple utility ...
> 

Either as it turns out or removing readme from pkg/ OK gsoares@

Looks like upstream merged a fix addressing the segfault, the PR was closed 
such that GH_COMMIT should be updated + get rid of patch-doh-c.

BR,
Gleydson.





Re: NEW: net/doh

2019-09-14 Thread Stuart Henderson
On 2019/09/13 12:57, Gleydson Soares wrote:
> i have fixed the segfault by adding a patch and also i put the pkg-readme.
> pull request done, upstream aware.
> 
> this one is OK gsoares@

I don't like hand-copying README.md to pkg/README at all.

README.md could be installed to PREFIX/share/doc/doh but it doesn't seem
very useful as it's right there on HOMEPAGE anyway, plus it's such a
simple utility ...



Re: NEW: net/doh

2019-09-13 Thread Gleydson Soares
i have fixed the segfault by adding a patch and also i put the pkg-readme.
pull request done, upstream aware.

this one is OK gsoares@


doh-revised.tgz
Description: application/tar-gz


Re: NEW: net/doh

2019-09-13 Thread Gleydson Soares
On Fri, Sep 13, 2019 at 02:48:22PM +0100, Stuart Henderson wrote:
> ---
> doh is a simple tool that uses libcurl to resolve a host name
> using a POST request to a DNS-over-HTTPS (DoH) server.
> ---
> 
> ok to import? (the syntax is friendlier than sdig for simple tests)
> 

buids and works fine here:

openbsd-dev$ doh trusted.com.br
trusted.com.br from https://dns.cloudflare.com/dns-query
TTL: 3064 seconds
A: 45.76.92.19

Noticed a bug though, seems argc is not well-handled:

openbsd-dev$ doh -t
Segmentation fault (core dumped)

Also, Would be useful installing the README as pkg-readme.


If DOH URL is left out, the Cloudflare DOH server will be used. See also
[list
of public
servers](https://github.com/curl/curl/wiki/DNS-over-HTTPS#publicly-available-servers)


:-)