Meterpreters Should No Longer Return TLV_TYPE_ADDR_TYPE When Resolving Hostnames

[zeroSteiner]

Now that https://github.com/rapid7/metasploit-framework/pull/17244 is landed, it's unnecessary for Meterepter's to return the TLV_TYPE_ADDR_TYPE when resolving hostnames. It appears to always be the value that was echoed to it and is redundant anways because the address is always 0 bytes long when it failed to resolve, 4 bytes long when it's IPv4 or 16 bytes long when it's IPv6. The information is redundant and should be omitted but that'd likely break backward compatibility so this ticket is our reminder.

[ArkaprabhaChakraborty]

I want to grab this issue too :)

[adfoster-r7]

Taking off the easy flag as this isn't a ticket that can be actively worked on due to it being a breaking change