Skip to content
This repository has been archived by the owner on Jun 24, 2022. It is now read-only.

Reinstate DNSCrypt #428

Merged
1 commit merged into from
Apr 12, 2018
Merged

Reinstate DNSCrypt #428

1 commit merged into from
Apr 12, 2018

Conversation

NPN
Copy link
Contributor

@NPN NPN commented Apr 4, 2018

Description

Add back DNSCrypt, as per discussion in #384. List of platforms has also been updated.

We may also want to change the logo, as the current one does not match the one on the website.

HTML Preview

http://htmlpreview.github.io/?https://github.com/NPN/privacytools.io/blob/reinstate-dnscrypt/index.html

@ghost ghost requested review from kewde and bookercodes April 4, 2018 13:14
@g-monk
Copy link

g-monk commented Apr 8, 2018

Question: As you guys know I’m new and learning, I wanted to ask why DNSCrypt was removed in the first place.

@ghost
Copy link

ghost commented Apr 8, 2018

I think it was not being maintained or something like that.

@zero77
Copy link

zero77 commented Apr 9, 2018

@NPN
I think DNSCrypt2 should be mentioned as it is fairly comprehensive now and it very stable.
Also it updated very regularly.

https://github.com/jedisct1/dnscrypt-proxy

@NPN
Copy link
Contributor Author

NPN commented Apr 11, 2018

@g-monk The issue I linked to above explains it. The basic reason is that v1 was dropped. Then, the website was redirected and eventually squatted. So, we removed it, because it seemed that the project might have been abandoned. Now, there's a new website and DNSCrypt-Proxy 2 is stable, so I think we should add it back.

@zero77 This change links to the DNSCrypt website, which has a big button that links you to "Implementations". DNSCrypt-Proxy 2 is at the top of that list. I think this is a better solution because it allows people learn about the project and then choose what implementation they want to use.

@ghost ghost merged commit ca0cbfe into privacytools:master Apr 12, 2018
@NPN NPN deleted the reinstate-dnscrypt branch April 12, 2018 19:46
Copy link
Contributor

@kewde kewde left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good.

@blacklight447 blacklight447 mentioned this pull request Feb 18, 2020
4 tasks
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants