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

Feature Request: Display hostname instead of ip addresses when force_ip_to_sysname is enabled #4715

Closed
4 tasks
breeman1 opened this issue Oct 4, 2016 · 7 comments

Comments

@breeman1
Copy link

breeman1 commented Oct 4, 2016

DO NOT DELETE THIS INFORMATION.

Please read this information carefully.

  • Is your install up to date? Updating your install
    Please do not submit an issue if your install is not up to date within the last 24 hours or on a stable monthly release.
  • Please include all of the information between the ==================================== section of ./validate.php.
  • If you would like us to add a new device then please provide the information asked for here
  • Please provide as much detail as possible.

There are various places in the GUI where I think displaying hostnames or sysnames instead of ip addresses would come in handy. Like when searching for an ip address as shown below, one might want to zoom in on a particular device whilst doing the search, but it not that easy remembering the ip addresses of all the devices particularly when you many on your install. The second scenario was when trying to edit User Permissions, being able to assign by sysname would be great as I have $config['force_ip_to_sysname'] = true; in my config file.

snap6
snap3
snap4

@laf
Copy link
Member

laf commented Oct 5, 2016

I'm confused by this, force_ip_to_sysname (whilst a confusing name) basically converts something to sysName, we only use that for hostname. So any device you add with a fqdn will just display that, if it's added by IP then we switch to use sysName instead of this flag is enabled.

So the title for this is confusing as we will never use hostname instead of IP as they are both the same and depend on how you add the device.

Do you mean that you have some areas of the web interface not honouring that feature?

@laf laf added the Needs-Info label Oct 5, 2016
@breeman1
Copy link
Author

breeman1 commented Oct 6, 2016

So most of our devices were added by ip and when we turned on the force_ip_to_sysname feature the ip were converted into hostnames in some areas of the GUI. Areas like the Availability-Map work as expected however as I showed using arrows in my previous post some areas of the GUI still diplay devices using ip addresses, the exception being only true if the devices were added bu hostname to begin with. I would have thought all areas where the device are listed should use the functionality if enabled of course. As an example, when searching for IPv4 addresses, in the pull-down menu in the top left hand side All devices display as ip addresses except those which were added by their respective hostnames. Likewise in the Edit User Permissions screen.

@laf
Copy link
Member

laf commented May 3, 2017

Can you provide areas that this still needs addressing in as some things have now been fixed.

@breeman1
Copy link
Author

breeman1 commented May 4, 2017

Hi @laf,

I am still experiencing this issue even on the latest code base;
image
User Permissions screen
image
IP address search screen
image

@laf
Copy link
Member

laf commented May 4, 2017

I realise it's still got issues, my question was for you to say which pages still have issues.

I've gone through and hopefully caught all the remainder - can you please test my linked pull request.

@breeman1
Copy link
Author

breeman1 commented May 5, 2017

It works @laf , well at least on the pages that I am aware of where it was not working. Thank you.

@lock
Copy link

lock bot commented May 16, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed.

@lock lock bot locked as resolved and limited conversation to collaborators May 16, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants