Comment 1 Vinícius Zavam 2017-10-24 14:27:41 UTC

Created attachment 187438 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

PORTREVISION++

I didn't like to speak up, but yes; alike minds...!
... as I have mixed feelings about Cisco AND the past and current administrations of the USA.

But in stead of using "random" I played with the thought of using 80.80.80.80 and 80.80.81.81 which are [a] anycasted (in surprising locations), and is [b] operated by myself (be it at an employer).
See http://80.80.80.80:80/

Let me try something that provides to set all three options (cisco, or random, or 80.80.80.80 & 80.80.81.81).

Comment 3 Vinícius Zavam 2017-10-25 14:09:14 UTC

(In reply to Leo Vandewoestijne from comment #2)

why don't we use 'random' for this new version? that sounds the easiest and cleaner solution for now.

people can change dnscrypt-proxy's config to use other resolver, if they want. you also see that we already have the possibility of running a "multi instances" of it and get more than one resolver available already.

using 'random' also ensures that the chosen resolver runs only IPV4 - as lots of users do not even have IPv6 (not even over a tunnel).

sorry to express myself this way, but sounds kind sad to see this postponed. anyway, it's your decision. if you have an account to update us the review D12775, would be pretty kind (I would like to follow your changes).

tyvm for your time and concern, Leo!

kindest regards,
vinícius

Created attachment 187583 [details]
update dnscrypt-proxy

(In reply to Vinícius Zavam from comment #3)

Don't worry. Attached patch is yours, plus also _multi modified.
Further also the old name "opendns" still appeared. And an accordingly updated pkg-message.


I foresee that not every individual user would embrace this change with joy, as OpenDNS clearly provides a great anycasted platform WITH distributed cache, and so performs significant faster than most others.
Those users should adjust their config.

Comment on attachment 187438 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

Thanks for contributing this idea and code.
It's merged into the new patch.

Comment 6 Vinícius Zavam 2017-11-06 14:01:02 UTC

Created attachment 187796 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

Comment 7 Vinícius Zavam 2017-11-06 14:02:41 UTC

(In reply to Leo Vandewoestijne from comment #5)

TYVM! Very appreciated. We actually forgot to update the pkg-messages and the URL used on the pkg-descr. I wrote the updates to this new patch.

I also updated the diff available at https://reviews.freebsd.org/D12775

KR,
vinícius

Comment on attachment 187796 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

Spot on. Hold on; I have some minor additions to this improvements.

Created attachment 187799 [details]
update dnscrypt-proxy and msg

Ah, I see, and also noticed that improvements you made could be done in the single files to.
So attached patch includes what you addressed (plus some minor changes of me).

Comment 10 Vinícius Zavam 2017-11-08 14:45:32 UTC

(In reply to Leo Vandewoestijne from comment #9)

hi,

I got some feedback from mat@, that says we should move some of the stuff related to "*_DESC" to a few lines before the options helpers - so I moved, and regenerated the patch (based on your new one).

`portlint -CA dns/dnscrypt-proxy` returns no error :)

thank you very much for all your concern, Leo! appreciated.

PS: aren't you on Phabricator? https://reviews.freebsd.org/D12775

KR,
vinícius

Comment 11 Vinícius Zavam 2017-11-08 14:46:49 UTC

Created attachment 187861 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

https://reviews.freebsd.org/D12775#270030

Comment 12 Vinícius Zavam 2017-11-13 11:47:35 UTC

Created attachment 187959 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

Following mat@'s and dbaio@'s recommendations, and request for changes, I:

  * added an entry to UPDATING;
  * updated MASTER_SITES following "https://marc.info/?l=openbsd-ports&m=151016589912210&w=2";
  * removed unnecessary tabulations/spaces.

backlog available on Phabricator.

KR,
Vinícius

Comment 13 Danilo G. Baio 2017-11-14 10:09:23 UTC

Comment on attachment 187959 [details]
[PATCH] dns/dnscrypt-proxy: replace 'cisco' resolver by 'random' (r452768)

last approval, please.

Comment 14 commit-hook 2017-12-23 14:16:30 UTC

A commit references this bug:

Author: dbaio
Date: Sat Dec 23 14:16:19 UTC 2017
New revision: 457054
URL: https://svnweb.freebsd.org/changeset/ports/457054

Log:
  dns/dnscrypt-proxy: Replace 'cisco' (OpenDNS) resolver by 'random'

  This update replaces the default resolver used by
  dnscrypt-proxy (cisco/OpenDNS) by a random one that
  does not log entries and supports dnssec.

  - Improve pkg-message's.
  - Fix MASTER_SITES.
  - Reorder Makefile variable's.
  - Update WWW.

  PR:		223222
  Submitted by:	Vin?cius Zavam <egypcio@googlemail.com>
  Reviewed by:	dbaio, garga, mat
  Approved by:	Leo Vandewoestijne <freebsd@dns-lab.com>
  		(maintainer, previous patch, then timeout)
  Differential Revision:	https://reviews.freebsd.org/D12775

Changes:
  head/UPDATING
  head/dns/dnscrypt-proxy/Makefile
  head/dns/dnscrypt-proxy/files/dnscrypt-proxy.in
  head/dns/dnscrypt-proxy/files/dnscrypt-proxy_multi.in
  head/dns/dnscrypt-proxy/files/pkg-message.in
  head/dns/dnscrypt-proxy/files/pkg-message_multi.in
  head/dns/dnscrypt-proxy/pkg-descr

Comment 15 Danilo G. Baio 2017-12-23 14:20:02 UTC

Committed, thank you both.