autodiscovery use dns name in description field

[zmon49]

Feature Request

Is your feature request related to a problem? Please describe

in previous versions of cacti, 1.2.10 specifically, auto discovery was able to discover the dns name of a device. 1.2.29 it still looks like its trying to do that, but doesn't find the DNS response in question (I am not sure if this is actually a bug, or dropped feature in recent release)
From there I was able to edit the PHP to tell it to default using the discovered dns name in the description field.

Describe the solution you'd like

Would like auto discovery to be able to find and use the DNS name of a device and use that in the description.

Describe alternatives you've considered

A clear and concise description of any alternative solutions or features you've
considered.

Additional context

Add any other context or screenshots about the feature request here.

[TheWitness]

Hostname right? Not host description.

[zmon49]

Description, and hostname really. but I made a separate request for hostname.
After some testing, I did figure out what's causing it not to find the DNS name.

There are 2 functions that are almost identical . this one works

cacti/lib/functions.php

Line 5984 in f04b942

function get_dns_from_ip($ip, $dns, $timeout = 1000) {

and this one doesnt.

cacti/lib/api_automation.php

Line 4247 in f04b942

function automation_get_dns_from_ip(string $ip, string $dns, int $timeout = 1000): string {

the auto discovery uses the non-working one by default.

The only difference seems to be the if statement that returns "return 'timed_out';" seems like the second one just always trues on that if statement. causing it to always return timed_out.