-
-
Notifications
You must be signed in to change notification settings - Fork 177
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
Fixed issue where ngrok cannot connect to a container if other argument #1047
base: develop
Are you sure you want to change the base?
Conversation
@sean-e-dietrich so that $container can be set like an ENV var? Should it be all caps then? |
@achekulaev i thought about that. Right now we have all lower case flags that are being used with Ngrok but we also allow for uppercase variables to be used. This is the one variable we just modified. So yes, we should allow someone to set it via a variable. |
Made so it would check against variables and see if NGROK_CONTAINER was previously set otherwise default to web
Ah it's a flag, I see. Thanks |
Needs to be rebased onto develop and tests updated/fixes (if necessary). |
I don't see any conflicts so rebasing doesn't look like a required step. Approving |
Can we just have this done as part of #1048 and ensure tests are green? |
Fixes #1044
This also makes it so if there is a different container other than web