-
Notifications
You must be signed in to change notification settings - Fork 176
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
feat: allow specifying port in robot url, so that user could use port other than 31950 #11820
Comments
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
This comment was marked as off-topic.
may I remind that issue is about supporting specifying a dnsname+port in the app? No difference for me if localhost is resolved or not, and I do not develop opentrons server. |
Hi @arogozhnikov, thanks for the request! We'd be happy to accept a PR for this issue. This is the location where the list of address strings from that UI is turned into a host + port pair. A solution that accounts for the optional presence of a colon seems like it would do the trick, as long as it accounts for IPv6 addresses: opentrons/app-shell/src/discovery.ts Lines 49 to 53 in 7a2e56a
|
Thanks for pointer @mcous I've tried to meaningfully contribute, but I wasn't able to install dev environment on mac. I have stable versions of nodejs and python, |
@arogozhnikov I no longer work at Opentrons, so I won't be able to be of much help, but I can tell you that much of the Opentrons development team works on macOS, so the development environment actually tends to work pretty well on mac. The development setup guide has pretty detailed instructions for setting up a development environment on macOS. If you're still interested in contributing, you might have some luck following that guide |
Overview
solutions for tunneling like cloudflared and ngrok do not provide freedom to choose which port used to expose service:
Now, app needs only one port to be exposed, so I'm running an additional redirection to get it working:
Now I can specify 127.0.0.1 as an address in opentrons app and it seems to work normally (robot is seen, I can operate with it).
Side comment: Interestingly, it understands IP, but does not understand
localhost
.I've tried to specify here both address and port (first line, partially covered name), but that does not work. Tried with http:https://, https:// and without protocol - no success.
That would be more convenient and more reliable than an additional SSH forward.
Importance: 3/5.
Implementation details
No response
Design
No response
Acceptance criteria
No response
The text was updated successfully, but these errors were encountered: