[PING] RFR 6425769: jmx remote bind address

[Severin Gehwolf]

Hi Jaroslav,

On Tue, 2015-12-01 at 12:33 +0100, Jaroslav Bachorik wrote:
> On 1.12.2015 11:17, Severin Gehwolf wrote:
> > On Mon, 2015-11-09 at 10:32 +0100, Severin Gehwolf wrote:
> > > On Wed, 2015-11-04 at 11:54 +0100, Severin Gehwolf wrote:
> > > > Hi,
> > > > 
> > > > Updated webrev with jtreg test in Java:
> > > > http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/02/
> > > > bug: https://bugs.openjdk.java.net/browse/JDK-6425769
> > > > 
> > > > I believe this updated webrev addresses all concerns and
> > > > incorporates
> > > > suggestions brought up by Jaroslav and Daniel.
> > > > 
> > > > I'm still looking for a sponsor and a hotspot/servicability-dev
> > > > reviewer. Could somebody maintaining javax.rmi.ssl have a look at
> > > > this
> > > > as well? Thank you!
> > > 
> > > Ping? Friendly reminder that I still need reviewers and a sponsor for
> > > this.
> > 
> > Anyone?
> 
> I'm sorry for not spotting this earlier:
> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/03.no-rmi-ssl-factory-changes/jdk/src/java.management/share/classes/sun/management/jmxremote/ConnectorBootstrap.java.sdiff.html
> * L442 - the log would contain 'com.sun.management.jmxremote.host = 
> null' if host is not specified; might be better not to print this out at all

Updated webrev which does not print
'com.sun.management.jmxremote.host = null' if unset:
http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-6425769/05/

> Other than that the change looks good to me. If no one else is 
> volunteering I may sponsor this change.

Thank you!

Cheers,
Severin

> Cheers,
> 
> -JB-