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

Removes ngrok as a tunneling option as this tool violates it's terms of service #35

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

Conversation

russorat
Copy link

@russorat russorat commented Nov 8, 2023

PM for ngrok here. This tool directly violates the ngrok Terms of Service even when used for educational purposes only. We kindly request that ngrok be removed as an option in your tool. Please consider replacing it with other options from this list.

To learn more about how ngrok combats abuse, see https://ngrok.com/abuse and https://ngrok.com/tos .

  • ngrok is not anonymous and can not be used to hide your identity.
  • ngrok directly exposes your public IP address to anyone who sees the ngrok url you send them and in an http header.
  • ngrok will ban your account if you use this tool.
  • ngrok adds an interstitial page to all requests warning anyone viewing the page that the site is hosted by ngrok.

This PR attempts to remove ngrok from this tool. I've also fixed a few spelling errors.

Signed-off-by: Russ Savage <[email protected]>
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.

None yet

1 participant