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

IP Address/Hostname of Octoprint instance should be shown #465

Closed
MatthewCroughan opened this issue Mar 1, 2020 · 8 comments
Closed

IP Address/Hostname of Octoprint instance should be shown #465

MatthewCroughan opened this issue Mar 1, 2020 · 8 comments
Labels
discussion Issue needs to be discussed further

Comments

@MatthewCroughan
Copy link

This will eliminate confusion when people want to access the web interface locally here at the makerspace. The value in my case would be 10.0.50.1 for the IP and matt-octoprint.local for the hostname of the octoprint instance. The problem is, these values do not match up with the values inside the config, so this has to be user programmable for vanity, as the address used in the config is literally octoprint since all of my configuration is making use of docker containers on the one device.

@MatthewCroughan MatthewCroughan added the enhancement New feature or request label Mar 1, 2020
@UnchartedBull UnchartedBull added discussion Issue needs to be discussed further and removed enhancement New feature or request labels Mar 2, 2020
@UnchartedBull
Copy link
Owner

Hmm, the problem I have with this is, that the value then would not match up with the actual config value, which could lead to a lot of confusion. From my point of view the config should reflect the actual config used by OctoDash ... Is there another reason for this, except that users might try to get the OctoPrint URL from the settings?

@MatthewCroughan
Copy link
Author

MatthewCroughan commented Mar 2, 2020

In docker, you can ping hosts with their container name, since that's how the routing works. If I execute ping octodash for example I'll get a response from the octodash container. So, in the OctoDash config I simply say the octoprint url is http://octoprint:80. This address and url is not accessible to other devices on the network, only the local device itself. Therefore it is not useful to display to users. All of this networking is internal and requires no router on the network. If you were to follow your idea of having the config use external addresses, OctoDash would break and be unable to communicate with the printer unless you owned a router, unless the config used 127.0.0.1, which would also be unhelpful to display.

@MatthewCroughan
Copy link
Author

MatthewCroughan commented Mar 2, 2020

Users who have no idea about any of this in the makerspace, perhaps even first timers to 3d printing in general should be able to see the external address that octoprint is accessible at in case they want to access it, that's all I'm suggesting. Currently I use the config.printer.name to display the printer hostname matt-ender3.local, but I'd also like to display an ip just in case their mdns isn't configured properly which can occur.

@UnchartedBull
Copy link
Owner

I totally get your point that users should know the public address. Changing the url field to display something different than what is actually being used currently is no option for me since then, if someone wants to change the actual ip or hostname he/she would need to edit the config.json file. Is there maybe an option for you to put something beside the printer that tells the users the IP. This could also be something for the help screen, but I would like to display the actual config values in the config menu. I'm thinking of a private/public mode here, which would show the settings for private and a help icon with markdown for public mode ...

@MatthewCroughan
Copy link
Author

Yeah there's no real way to automatically get this, as in my circumstance the device would be running in container and would think its IP address is 172.xx.xx.xx and hostname of octodash.

@UnchartedBull
Copy link
Owner

Then there is also no easy way for OctoDash to get the correct IP of OctoPrint ...

@MatthewCroughan
Copy link
Author

Misclicked on my phone, didn't mean to close the issue.

@UnchartedBull
Copy link
Owner

I decided against showing another IP in the settings. There already is some confusion around this, so showing something different than is actually being used could create even more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Issue needs to be discussed further
Projects
None yet
Development

No branches or pull requests

2 participants