docs icon indicating copy to clipboard operation
docs copied to clipboard

Improve documentation of needed ports

Open DL6ER opened this issue 5 years ago • 5 comments

Multi-purpose PR to improve the documentation about the necessary ports for Pi-hole:

  • FTL only uses port 4711 for its telnet API (for quite some time)
  • Add open resolver warning below the table of necessary ports to remind users that this table does NOT (as in: NEVER) read this table as a list of ports to open from the WWW onto their Pi-holes. This is just bad idea. Fullstop.
  • Add a section about why we use random UDP ports by default (can be disabled, but we don't say this explicitly here)

This PR is independent of the FTL version and be merged right away.

DL6ER avatar Aug 21 '20 14:08 DL6ER

Deploy Preview for pihole-docs ready!

Name Link
Latest commit 79a55158323e3e7b08b97849150e1019fd4b303b
Latest deploy log https://app.netlify.com/sites/pihole-docs/deploys/630bad4c20140d00082139fc
Deploy Preview https://deploy-preview-366--pihole-docs.netlify.app/main/prerequisites
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

netlify[bot] avatar Aug 21 '20 14:08 netlify[bot]

I could need some help with the broken tests here, I did not touch any of these links:

Screenshot at 2020-08-21 16-53-45

Furthermore, I checked for the first link in here and this site seems to be linked to from no page at all expect the navigation menu:

$ grep "github/contri" -Ri *
mkdocs.yml:    - 'Contributing on GitHub': guides/github/contributing.md

So why is this error even there?

DL6ER avatar Aug 21 '20 15:08 DL6ER

There are merge conflicts here in package.json, else LGTM

PromoFaux avatar Dec 31 '20 13:12 PromoFaux

Rebased on latest master

DL6ER avatar Dec 31 '20 14:12 DL6ER

Converting again to draft so we can update the recommendation from OpenVPN to Wireguard once the other PR is merged.

DL6ER avatar Dec 31 '20 14:12 DL6ER

Rebased on latest master and changed VPN recommendation from OpenVPN to Wireguard

DL6ER avatar Aug 21 '22 07:08 DL6ER

This PR is considered stale.

DL6ER avatar Jan 28 '23 11:01 DL6ER