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

Documentation Issue: Dns-Over-HTTPs with Cloudflared #175

Open
3 tasks done
iUnknwn opened this issue Oct 7, 2019 · 6 comments
Open
3 tasks done

Documentation Issue: Dns-Over-HTTPs with Cloudflared #175

iUnknwn opened this issue Oct 7, 2019 · 6 comments

Comments

@iUnknwn
Copy link
Contributor

iUnknwn commented Oct 7, 2019

In raising this issue, I confirm the following: {please fill the checkboxes, e.g: [X]}

How familiar are you with the the source code relevant to this issue?:

N/A - Documentation Issue


Expected behaviour:

Documentation on Encrypted DNS via Cloudflared should use correct systemd paths, and provide update instructions.

Actual behaviour:

Documentation instructs users to update the system built-in unit store, instead of the user config store. No update instructions provided.

Steps to reproduce:

N/A - however, issues can be fixed by replacing the systemd unit /lib/systemd/system path with /etc/systemd/system/, which is the user-created store for systemd files.

Also, while this is slightly more of a documentation change request, there's no information on updating the cloudflared daemon. While not exactly elegant, something like this could be called as a cron job:

echo "Updating Cloudflared Daemon..."
wget -P /tmp https://bin.equinox.io/c/VdrWdbjqyF/cloudflared-stable-linux-arm.tgz
systemctl stop cloudflared
tar -C /opt/cloudflared -xzf /tmp/cloudflared-stable-linux-arm.tgz
systemctl start cloudflared
rm /tmp/cloudflared-stable-linux-arm.tgz
echo "Cloudflared update complete."

Debug token provided by uploading pihole -d log:

N/A

Troubleshooting undertaken, and/or other relevant information:

N/A

@dschaper dschaper transferred this issue from pi-hole/pi-hole Oct 7, 2019
@PromoFaux
Copy link
Member

Hi @iUnknwn , please feel free to submit any changes to the documentation as a Pull request on this repository.

The team can look over your suggested changes as part of the entire document, then.

https://github.com/pi-hole/docs

@DL6ER
Copy link
Member

DL6ER commented Oct 7, 2019

Also note that this is a community-provided guide initially written by @bendews (#23) and later updated/extended by others. They might want to comment on this as well.

@XhmikosR
Copy link
Contributor

XhmikosR commented Dec 3, 2019

I was actually looking at this myself, and it seems cloudflared has a service command https://developers.cloudflare.com/argo-tunnel/reference/arguments/#service-command

So, it could be possible that the whole manual thing could be simplified nowadays.

@iUnknwn
Copy link
Contributor Author

iUnknwn commented Jan 16, 2020

I submitted a small pull request that expands the docs, so a user can setup automatic updating of the cloudflared daemon. It looks like someone already updated the docs with the corrected systemd path.

@dschaper
Copy link
Member

I was actually looking at this myself, and it seems cloudflared has a service command

Screenshot_2020-01-15 Command-line Arguments - Argo Tunnel

@XhmikosR
Copy link
Contributor

@dschaper yeah I have this in #186

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants