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 its terms of service #316

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

Conversation

russorat
Copy link

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.

Signed-off-by: Russ Savage <russ@ngrok.com>
Copy link

Choose a reason for hiding this comment

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

IS THE CODE READY?

@joshkar
Copy link
Contributor

joshkar commented Apr 30, 2024

Hello, I will consult with the development team about the message you posted, thank you

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

3 participants