search

jcktej / jdiameter

Automatically exported from code.google.com/p/jdiameter
0 stars 0 forks source link

JDiameter replace getHostName with getHostString #58

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
jDiameter often uses InetSocketAddress.getHostName - this does a reverse DNS 
name
If you have many diameter peers and they do not have reverse DNS entries this 
reverse DNS lookup can take a long time to fail and cause problems.

Instead of InetSocketAddress.getHostName use InetSocketAddress.getHostString

Patched attached against jDiameter master.

Original issue reported on code.google.com by richard....@smilecoms.com on 21 May 2014 at 1:23

Attachments:

GoogleCodeExporter commented 9 years ago 
Thanks for the report Richard, indeed it is a good suggestion, but that would 
make us JDK7 dependent, which we are not (yet).. in fact, I think our stack can 
still be compiled with JDK5.

I don't have any strong opposition to that, but for instance I cannot run AS5 
with Oracle JDK7, not sure what's the situation with other vendors JVM.

I will keep this targeted to a later time (1.7.0.FINAL).

Original comment by brainslog on 4 Jul 2014 at 11:40