Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

dnsdist: add documentation links to dnsdist.service #3714

Merged
merged 1 commit into from Apr 22, 2016

Conversation

@rubenk
Copy link
Contributor

@rubenk rubenk commented Apr 15, 2016

No description provided.

@@ -1,5 +1,7 @@
[Unit]
Description=DNS Loadbalancer
Documentation=man:dnsdist(1)
Documentation=https://dnsdist.org

This comment has been minimized.

@pieterlexis

pieterlexis Apr 16, 2016
Member

the dnsdist website does not have https.

This comment has been minimized.

@rubenk

rubenk Apr 16, 2016
Author Contributor

It does, with a self-signed certificate ;)

I'll fix it up.

@pieterlexis
Copy link
Member

@pieterlexis pieterlexis commented Apr 16, 2016

one nit, awesome otherwise

@rgacogne rgacogne added the dnsdist label Apr 18, 2016
@rgacogne rgacogne added this to the dnsdist-1.0.0 milestone Apr 18, 2016
@rgacogne
Copy link
Member

@rgacogne rgacogne commented Apr 19, 2016

👍 except for the https link :-)

@rgacogne rgacogne modified the milestones: dnsdist-1.1.0, dnsdist-1.0.0 Apr 20, 2016
@rubenk rubenk force-pushed the rubenk:dnsdist-service branch from d97743c to 06b2f3b Apr 21, 2016
@rubenk
Copy link
Contributor Author

@rubenk rubenk commented Apr 21, 2016

Fixed.

@rgacogne rgacogne merged commit 722186f into PowerDNS:master Apr 22, 2016
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@rubenk rubenk deleted the rubenk:dnsdist-service branch Apr 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants