Return-Path: X-Original-To: apmail-jmeter-dev-archive@minotaur.apache.org Delivered-To: apmail-jmeter-dev-archive@minotaur.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id 3ABE4182EB for ; Wed, 30 Dec 2015 09:43:10 +0000 (UTC) Received: (qmail 79860 invoked by uid 500); 30 Dec 2015 09:43:09 -0000 Delivered-To: apmail-jmeter-dev-archive@jmeter.apache.org Received: (qmail 79809 invoked by uid 500); 30 Dec 2015 09:43:09 -0000 Mailing-List: contact dev-help@jmeter.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@jmeter.apache.org Delivered-To: mailing list dev@jmeter.apache.org Received: (qmail 79788 invoked by uid 99); 30 Dec 2015 09:43:09 -0000 Received: from mail-relay.apache.org (HELO mail-relay.apache.org) (140.211.11.15) by apache.org (qpsmtpd/0.29) with ESMTP; Wed, 30 Dec 2015 09:43:09 +0000 Received: from ok2c (84.76.106.92.dynamic.wline.res.cust.swisscom.ch [92.106.76.84]) by mail-relay.apache.org (ASF Mail Server at mail-relay.apache.org) with ESMTPSA id C05F61A0015; Wed, 30 Dec 2015 09:43:08 +0000 (UTC) Message-ID: <1451468581.20141.10.camel@apache.org> Subject: Re: About SNI support in HC4.5 From: Oleg Kalnichevski To: HttpClient User Discussion Cc: "dev@jmeter.apache.org" Date: Wed, 30 Dec 2015 10:43:01 +0100 In-Reply-To: References: <1450877979.7429.2.camel@apache.org> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.12.9-1+b1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit On Tue, 2015-12-29 at 09:22 +0100, Philippe Mouawad wrote: > Hello Oleg, > Thanks for answer. > Could you elaborate a bit more on your answer ? > > Also, is there a chance that you fix the issue also in deprecated code of > httpclient (org.apache.http.conn.ssl.SSLSocketFactory) or is it too complex > ? > I personally see no point in fixing non-critical bugs in deprecated code. This time can be better spent elsewhere. However I can commit a fix though if contributed. Oleg > Thanks > > On Wednesday, December 23, 2015, Oleg Kalnichevski wrote: > > > On Wed, 2015-12-23 at 14:30 +0100, Philippe Mouawad wrote: > > > Hi, > > > Any answer on the 1/ question ? > > > Thanks > > > Regards > > > > > > On Tue, Dec 22, 2015 at 2:39 PM, Philippe Mouawad < > > > philippe.mouawad@gmail.com > wrote: > > > > > > > Hi, > > > > With further investigation, I answered to some of my questions. > > > > > > > > Regards > > > > > > > > On Mon, Dec 21, 2015 at 6:52 PM, Philippe Mouawad < > > > > philippe.mouawad@gmail.com > wrote: > > > > > > > >> Hello, > > > >> Looking at: > > > >> > > > >> - https://issues.apache.org/jira/browse/HTTPCLIENT-1119 > > > >> > > > >> > > > >> 1/ It is not clear for me how the fix works if commit association is > > > >> correct: > > > >> > > > >> - > > https://fisheye6.atlassian.com/changelog/httpcomponents?cs=1544769 > > > >> > > > >> Is the fix about replacing: > > > >> > > > >> final SSLSocket sock = (SSLSocket) this.socketfactory.createSocket(); > > > >> > > > >> with : > > > >> SocketFactory.getDefault().createSocket(); > > > > Pretty much. Please note though that the use of a plain socket instead > > of a fully initialized SSL socket causes a different execution path in > > #connectSocket method. > > > > Oleg > > > > > > > > >