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

Update README.md #1307

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #1307

wants to merge 1 commit into from

Conversation

Yoxmo
Copy link

@Yoxmo Yoxmo commented Aug 6, 2024

minor fixes to readme

minor fixes to readme
@@ -1,10 +1,10 @@
# Cloudflare Tunnel client

Contains the command-line client for Cloudflare Tunnel, a tunneling daemon that proxies traffic from the Cloudflare network to your origins.
This daemon sits between Cloudflare network and your origin (e.g. a webserver). Cloudflare attracts client requests and sends them to you
via this daemon, without requiring you to poke holes on your firewall --- your origin can remain as closed as possible.
This daemon sits between Cloudflared network and your origin (e.g. a webserver). Cloudflare attracts client requests and sends them to you
Copy link

Choose a reason for hiding this comment

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

"Cloudflare" should be more precise than Cloudflared
As Cloudflared is just a daemon connect to Cloudflare network. Cloudflared is not a network

Copy link
Contributor

Choose a reason for hiding this comment

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

Hi @longz3r, by Cloudflare network, we mean cloudflared is between Cloudflare's edge network and your origin. We would prefer to keep the documentation this way. The rest of your changes looks good though. We are happy to accept this PR if you can revert this line.

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

Successfully merging this pull request may close these issues.

3 participants