Re: Adding SocketChannel toString to connection exception messages

2018-01-03 Thread David Lloyd
On Fri, Dec 29, 2017 at 11:15 AM, Chris Hegarty wrote: > On 29 Dec 2017, at 00:33, Steven Schlansker > wrote: >> Thanks for the discussion! >> >> So, it sounds like amending the message by default is going to be a >> non-starter -- but at least adding the information otherwise might be >> poss

Re: Adding SocketChannel toString to connection exception messages

2018-01-03 Thread Alan Bateman
On 02/01/2018 21:25, Steven Schlansker wrote: : This would definitely be better than nothing! But it's still difficult, for example a common allocation pattern for us would be to assign networks to availability zones: 10.0.1.0/24 10.0.2.0/24 10.0.3.0/24 then if you pick the same last number

Re: Adding SocketChannel toString to connection exception messages

2018-01-02 Thread Peter Levart
Hi Steven, Steven Schlansker je 02. 01. 2018 ob 22:25 napisal: On Jan 2, 2018, at 8:35 AM, Alan Bateman wrote: On 29/12/2017 00:33, Steven Schlansker wrote: Thanks for the discussion! So, it sounds like amending the message by default is going to be a non-starter -- but at least adding the

Re: Adding SocketChannel toString to connection exception messages

2018-01-02 Thread Steven Schlansker
On Dec 31, 2017, at 7:24 AM, Peter Levart wrote: > > Hi, >> >> I believe there are concerns with too much information that can be >> considered "sensitive" (like host names and IP addresses) appearing in error >> messages due to them ending up in log files and bug reports. >> >> David > > F

Re: Adding SocketChannel toString to connection exception messages

2018-01-02 Thread Alan Bateman
On 29/12/2017 00:33, Steven Schlansker wrote: Thanks for the discussion! So, it sounds like amending the message by default is going to be a non-starter -- but at least adding the information otherwise might be possible. There are examples in other area where exceptions include detail informa

Re: Adding SocketChannel toString to connection exception messages

2017-12-31 Thread Peter Levart
Hi, David Holmes je 22. 12. 2017 ob 01:35 napisal: On 22/12/2017 10:29 AM, Steven Schlansker wrote: On Dec 21, 2017, at 11:11 AM, Steven Schlansker wrote: What if ConnectException included the attempted hostname / IP / port SocketAddress? java.net.ConnectException: Connection to 'foo.myc

Re: Adding SocketChannel toString to connection exception messages

2017-12-29 Thread Chris Hegarty
n behalf of Chris Hegarty >> >> Sent: Friday, December 22, 2017 4:17:31 PM >> To: Seán Coffey; David Holmes; Steven Schlansker >> Cc: core-libs-dev; net-...@openjdk.java.net >> Subject: Re: Adding SocketChannel toString to connection exception messages >> >> >>>

Re: Adding SocketChannel toString to connection exception messages

2017-12-28 Thread Steven Schlansker
gt; Bernd > -- > http://bernd.eckenfels.net > > From: net-dev on behalf of Chris Hegarty > > Sent: Friday, December 22, 2017 4:17:31 PM > To: Seán Coffey; David Holmes; Steven Schlansker > Cc: core-libs-dev; net-...@openjdk.java.net > Subject: Re: Add

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread Bernd Eckenfels
. Gruss Bernd -- http://bernd.eckenfels.net From: net-dev on behalf of Chris Hegarty Sent: Friday, December 22, 2017 4:17:31 PM To: Seán Coffey; David Holmes; Steven Schlansker Cc: core-libs-dev; net-...@openjdk.java.net Subject: Re: Adding SocketChannel toString to

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread Remi Forax
d Holmes" > Cc: "core-libs-dev" > Envoyé: Vendredi 22 Décembre 2017 16:22:41 > Objet: Re: Adding SocketChannel toString to connection exception messages > On Thu, Dec 21, 2017 at 6:35 PM, David Holmes wrote: >> I believe there are concerns with too much informati

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread David Lloyd
On Thu, Dec 21, 2017 at 6:35 PM, David Holmes wrote: > I believe there are concerns with too much information that can be > considered "sensitive" (like host names and IP addresses) appearing in error > messages due to them ending up in log files and bug reports. I tend to agree here. However -

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread Chris Hegarty
On 22/12/17 14:59, Seán Coffey wrote: As someone who works with alot of log files, I'd like to chime in and support Steven's end goal. Looking at a "Connection refused" error in the middle of a log file that possibly extends to millions of lines is near useless. In the era of cloud compute, di

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread Seán Coffey
As someone who works with alot of log files, I'd like to chime in and support Steven's end goal. Looking at a "Connection refused" error in the middle of a log file that possibly extends to millions of lines is near useless. In the era of cloud compute, diagnosing network issues is sure to beco

Re: Adding SocketChannel toString to connection exception messages

2017-12-22 Thread Chris Hegarty
On 22/12/17 01:27, David Holmes wrote: cc'ing net-dev as that might be the more appropriate list. On 22/12/2017 10:59 AM, Steven Schlansker wrote: On Dec 21, 2017, at 4:35 PM, David Holmes wrote: On 22/12/2017 10:29 AM, Steven Schlansker wrote: On Dec 21, 2017, at 11:11 AM, Steven Schlansk

Re: Adding SocketChannel toString to connection exception messages

2017-12-21 Thread David Holmes
cc'ing net-dev as that might be the more appropriate list. On 22/12/2017 10:59 AM, Steven Schlansker wrote: On Dec 21, 2017, at 4:35 PM, David Holmes wrote: On 22/12/2017 10:29 AM, Steven Schlansker wrote: On Dec 21, 2017, at 11:11 AM, Steven Schlansker wrote: What if ConnectException in

Re: Adding SocketChannel toString to connection exception messages

2017-12-21 Thread Steven Schlansker
> On Dec 21, 2017, at 4:35 PM, David Holmes wrote: > > On 22/12/2017 10:29 AM, Steven Schlansker wrote: >>> On Dec 21, 2017, at 11:11 AM, Steven Schlansker >>> wrote: >>> >>> What if ConnectException included the attempted hostname / IP / port >>> SocketAddress? >>> java.net.ConnectException

Re: Adding SocketChannel toString to connection exception messages

2017-12-21 Thread David Holmes
On 22/12/2017 10:29 AM, Steven Schlansker wrote: On Dec 21, 2017, at 11:11 AM, Steven Schlansker wrote: What if ConnectException included the attempted hostname / IP / port SocketAddress? java.net.ConnectException: Connection to 'foo.mycorp.com[10.x.x.x]:12345' refused Much more useful! T

Re: Adding SocketChannel toString to connection exception messages

2017-12-21 Thread Steven Schlansker
> On Dec 21, 2017, at 11:11 AM, Steven Schlansker > wrote: > > What if ConnectException included the attempted hostname / IP / port > SocketAddress? > java.net.ConnectException: Connection to 'foo.mycorp.com[10.x.x.x]:12345' > refused > Much more useful! This could also be extended to variou