Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Gary D. Gregory
Closing the loop, FTR https://issues.apache.org/jira/browse/HTTPCORE-729 Gary On 2022/11/14 14:31:43 Gary Gregory wrote: > I will provide a fix for the NPE. Ah, now I see you did. > > I will eventually update VFS to avoid deprecation warnings. > > This issue was more about testing HC 5.2 as a d

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Gary Gregory
I will provide a fix for the NPE. Ah, now I see you did. I will eventually update VFS to avoid deprecation warnings. This issue was more about testing HC 5.2 as a drop in replacement for 5.1.x, which it isn't, but 5.2.1 should be. Gary On Mon, Nov 14, 2022, 09:10 Oleg Kalnichevski wrote: > On

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Oleg Kalnichevski
On Mon, 2022-11-14 at 13:51 +, Gary D. Gregory wrote: > On 2022/11/14 13:17:53 Oleg Kalnichevski wrote: > > On Mon, 2022-11-14 at 13:10 +, Gary D. Gregory wrote: > > > What did you test exactly? Note that running the 5.2 async > > > example > > > from the HC repo is _not_ the test to perfor

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Gary D. Gregory
On 2022/11/14 13:17:53 Oleg Kalnichevski wrote: > On Mon, 2022-11-14 at 13:10 +, Gary D. Gregory wrote: > > What did you test exactly? Note that running the 5.2 async example > > from the HC repo is _not_ the test to perform. > > > > The failure is not specific to my environment since the PR t

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Oleg Kalnichevski
On Mon, 2022-11-14 at 13:10 +, Gary D. Gregory wrote: > What did you test exactly? Note that running the 5.2 async example > from the HC repo is _not_ the test to perform. > > The failure is not specific to my environment since the PR to update > the HC version also fails on GitHub: > https://

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Gary D. Gregory
What did you test exactly? Note that running the 5.2 async example from the HC repo is _not_ the test to perform. The failure is not specific to my environment since the PR to update the HC version also fails on GitHub: https://github.com/apache/commons-vfs/pull/332 The failure is not apparent

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-14 Thread Oleg Kalnichevski
On Sun, 2022-11-13 at 13:45 +, Gary D. Gregory wrote: > So, logging using 5.1.3 looks like this: > > [INFO] --- exec-maven-plugin:3.1.0:java (default-cli) @ commons-vfs2 > --- > Sun, 13 Nov 2022 13:41:22 GMT | Serving . on /0:0:0:0:0:0:0:0:51729 > Sun, 13 Nov 2022 13:41:38 GMT | Handling [head

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-13 Thread Gary D. Gregory
So, logging using 5.1.3 looks like this: [INFO] --- exec-maven-plugin:3.1.0:java (default-cli) @ commons-vfs2 --- Sun, 13 Nov 2022 13:41:22 GMT | Serving . on /0:0:0:0:0:0:0:0:51729 Sun, 13 Nov 2022 13:41:38 GMT | Handling [head=GET http://localhost:51729/pom.xml, body=null] in org.apache.hc.cor

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-13 Thread Gary D. Gregory
I added a logging event at the very start of the handle() method. Gary On 2022/11/13 13:24:19 "Gary D. Gregory" wrote: > On 2022/11/13 12:58:41 Oleg Kalnichevski wrote: > > On Sat, 2022-11-12 at 23:21 +, Gary D. Gregory wrote: > > > Hi All: > > > > > > I am trying to bump HC from 5.1.3 to 5.

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-13 Thread Gary D. Gregory
On 2022/11/13 12:58:41 Oleg Kalnichevski wrote: > On Sat, 2022-11-12 at 23:21 +, Gary D. Gregory wrote: > > Hi All: > > > > I am trying to bump HC from 5.1.3 to 5.2 in Apache Commons VFS and I > > get an odd failure. > > > > To reproduce: > > > > git clone https://gitbox.apache.org/repos/asf

Re: Possible regression in HttpClient 5.2 vs 5.1.3

2022-11-13 Thread Oleg Kalnichevski
On Sat, 2022-11-12 at 23:21 +, Gary D. Gregory wrote: > Hi All: > > I am trying to bump HC from 5.1.3 to 5.2 in Apache Commons VFS and I > get an odd failure. > > To reproduce: > > git clone https://gitbox.apache.org/repos/asf/commons-vfs.git > cd commons-vfs > > Then, this works: > > mvn