View Issue Details
|ID||Project||Category||View Status||Date Submitted||Last Update|
|0003437||unreal||ircd||public||2007-07-12 13:37||2007-11-24 23:26|
|Target Version||Fixed in Version||4.0-devel|
|Summary||0003437: localhost should be cloaked|
|Description||Brain insists it's invalid, but I don't like people on my network knowing someone is connecting from a localhost... This creates issues like "zomg, I want access to use IRC too!"|
Unlike Unreal 3.*, 4 should probably cloak the actual localhost IP in instead of "localhost" itself.
|Tags||No tags attached.|
|3rd party modules|
Indeed, I don't much like doing special case stuff like this. But just at looking at the code, it almost seems like it's because the cloak module is just doing a "find the domain by top two parts, or is the whole thing if less than two parts". Which really isn't the correct way to find out what the domain is. In reality it should go more like:
localhost => clk-ABEHFLJ # Also means if we had a private DNS server that resolved LAN ips as simple names (eg 192.168.2.34 -> "bob") they get cloaked correctly.
server.com => clk-ABKJLD.com # Hide the server if there's no subdomain.
sub.server.com => clk-lkdfl.server.com # With subdomain, leave main server name alone.
inter.national.server.co.uk => clk-sfjklsdjf.server.co.uk # Intelligently know how deep a TLD goes, like .co.uk etc.
220.127.116.11 => lkjf.dfsdkfj.sfsd.IP # IP, module doesn't do it like this, of course. It should, since that's the unreal way ;) .
||This got fixed in revision 242 (there was a check copy-pasted from Insp m_cloaking.cpp that still kept it from cloaking one-part hosts which was removed in the process of fixing something else).|
|2007-07-12 13:37||Stealth||New Issue|
|2007-07-13 15:03||aquanight||Relationship added||child of 0003417|
|2007-07-13 15:13||aquanight||Note Added: 0014479|
|2007-07-13 15:13||aquanight||Status||new => confirmed|
|2007-11-24 23:26||aquanight||QA||=> Fixed by developer, needs QA testing|
|2007-11-24 23:26||aquanight||U4: Need for upstream patch||=> No need for upstream InspIRCd patch|
|2007-11-24 23:26||aquanight||U4: Upstream notification of bug||=> Not decided|
|2007-11-24 23:26||aquanight||U4: Contributor working on this||=> None|
|2007-11-24 23:26||aquanight||Note Added: 0014831|
|2007-11-24 23:26||aquanight||Assigned To||=> aquanight|
|2007-11-24 23:26||aquanight||Status||confirmed => resolved|
|2007-11-24 23:26||aquanight||Resolution||open => fixed|
|2007-11-24 23:26||aquanight||Fixed in Version||=> 4.0-devel|