-
Notifications
You must be signed in to change notification settings - Fork 317
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
Time for deprecation?... #319
Comments
The new, official (binary-free) package: @ngrok/ngrok |
It seems like in the official package it is not possible to use the ngrok from command line. node_modules/@ngrok/ngrok/bin/ngrok |
@scr4bble Why is running Ngrok from the command line a requirement? The whole point of a package like this is to integrate Ngrok into your Node.js app. Here's a working example built for Sails.js apps: Using that script, I can just run |
Not a requirement for everyone but a feature that some people might need, including us as it allows us to automate a few things and make use of the local web interface amongst others - see ngrok/ngrok-javascript#62 |
You kind of explained it inside your reply. The file you linked to is exactly the thing we don't need when being able to just run the "binary". Our
That feels much easier than being forced to write the javascript/typescript code to integrate ngrok into the app. |
how about just stfu if you don't use the command line |
While myself (and I'm sure countless others) appreciate what this package did for us in the past; with Ngrok's new official NPM package (which clocks in less than a MB after secondary installs), I'm left asking the question, is it time to retire / deprecate this package?
It seems like a LOT of work moving forward otherwise.
The text was updated successfully, but these errors were encountered: