Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-28 Thread Dan Tran 
strange, i placed your recommended logger settings
under systemPropertyVariables

and see no log for the timeout issue

Anyway I am going to cancel the vote and have to roll it again

Thanks

-Dan

On Wed, Dec 28, 2016 at 6:18 AM, Michael Osipov  wrote:

> Am 2016-12-28 um 15:07 schrieb Olivier Lamy:
>
>> 2017 is in few days so maybe java 1.7 shouldn't be a big problem
>>
>
> While this is correct, I don't want to change 10 things at once. At last,
> if no one changes the code to 1.7, there won't be any benefit. I'd move
> this for n+1 release.
>
> Michael
>
> On Wed, 28 Dec 2016 at 11:09 am, Michael Osipov 
>> wrote:
>>
>> Am 2016-12-28 um 08:54 schrieb Olivier Lamy:
>>
>> Hi
>>>
>>
>> What about at least Jetty 9.2? (8.x is not anymore supported)
>>>
>>
>>
>>
>> Wagon is still Java 1.6, therefore I had to stick to Jetty 8.
>>
>> Moving from 8 to 9 is easier than from 6 to 9.
>>
>>
>>
>> See:
>>
>> https://www.eclipse.org/jetty/documentation/9.3.x/what-jetty-version.html
>>
>>
>>
>> On 26 December 2016 at 10:58,  wrote:
>>>
>>
>>
>>>
>> Repository: maven-wagon

>>>
>> Updated Branches:

>>>
>>   refs/heads/jetty-8 [created] e707a2691

>>>
>>

>>

>> [WAGON-474] Upgrade and revise all tests for Jetty 8

>>>
>>

>> * Upgrade all test code to Jetty 8.1.22 and Servlet 3.0

>>>
>> * Unify variable names in redirect usecases to realServer and

>>>
>>   redirectServer

>>>
>> * RedirectHandler: redirect code is passed but completely ignored because

>>>
>>   sendRedirect() always sends 302

>>>
>> * Chronologically sort checkHandlerResult() calls

>>>
>> ** Set redirect code (See Other (303)) as requested

>>>
>> ** Update checkHandlerResult() for requested status codes rather sent

>>>
>>chosen by server (mismatched previously)

>>>
>> * testPreemptiveAuthentication*(): properly check for OK for GET and

>>>
>>   CREATED for PUT instead of OK only for both

>>>
>> * WebDavWagonTest: replace status code literal for

>>>
>>   HttpServletResponse.SC_* for better readability

>>>
>> * testRedirect*(): add more checkHandlerResults

>>>
>>

>>

>> Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo

>>>
>> Commit:

>>> http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e
>>
>> Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e

>>>
>> Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e

>>>
>>

>> Branch: refs/heads/jetty-8

>>>
>> Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee

>>>
>> Parents: b451e41

>>>
>> Author: Michael Osipov 

>>>
>> Authored: Mon Dec 26 00:55:09 2016 +0100

>>>
>> Committer: Michael Osipov 

>>>
>> Committed: Mon Dec 26 00:55:09 2016 +0100

>>>
>>

>> --

>>>
>>  pom.xml |  14 +-

>>>
>>  wagon-provider-test/pom.xml |   8 +-

>>>
>>  .../maven/wagon/http/HttpWagonTestCase.java | 273

>>> ++-
>>
>>  wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-

>>>
>>  .../http/LightweightHttpsWagonTest.java |   6 +-

>>>
>>  wagon-providers/wagon-http/pom.xml  |   8 +-

>>>
>>  .../http/HttpWagonHttpServerTestCase.java   |  14 +-

>>>
>>  .../http/HttpWagonReasonPhraseTest.java |   2 +-

>>>
>>  .../providers/http/HttpWagonTimeoutTest.java|   2 +-

>>>
>>  .../http/HttpsWagonPreemptiveTest.java  |   6 +-

>>>
>>  .../wagon/providers/http/HttpsWagonTest.java|   6 +-

>>>
>>  .../providers/http/HugeFileDownloadTest.java|  14 +-

>>>
>>  wagon-providers/wagon-ssh/pom.xml   |   8 +-

>>>
>>  .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-

>>>
>>  wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-

>>>
>>  .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---

>>>
>>  .../providers/webdav/WebDavsWagonTest.java  |   6 +-

>>>
>>  wagon-tcks/wagon-tck-http/pom.xml   |  12 +-

>>>
>>  .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--

>>>
>>  19 files changed, 278 insertions(+), 254 deletions(-)

>>>
>> --

>>>
>>

>>

>> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml

>>>
>> --

>>>
>> diff --git a/pom.xml b/pom.xml

>>>
>> index e127da4..a90a1ba 100644

>>>
>> --- a/pom.xml

>>>
>> +++ b/pom.xml

>>>
>> @@ -313,14 +313,14 @@ under the License.

>>>
>>

>>>
>>

>>

>>>
>> -org.mortbay.jetty

>>>
>> -

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-28 Thread Michael Osipov 

Am 2016-12-28 um 15:07 schrieb Olivier Lamy:

2017 is in few days so maybe java 1.7 shouldn't be a big problem


While this is correct, I don't want to change 10 things at once. At 
last, if no one changes the code to 1.7, there won't be any benefit. I'd 
move this for n+1 release.


Michael


On Wed, 28 Dec 2016 at 11:09 am, Michael Osipov  wrote:

Am 2016-12-28 um 08:54 schrieb Olivier Lamy:


Hi



What about at least Jetty 9.2? (8.x is not anymore supported)




Wagon is still Java 1.6, therefore I had to stick to Jetty 8.

Moving from 8 to 9 is easier than from 6 to 9.



See:

https://www.eclipse.org/jetty/documentation/9.3.x/what-jetty-version.html




On 26 December 2016 at 10:58,  wrote:







Repository: maven-wagon



Updated Branches:



  refs/heads/jetty-8 [created] e707a2691











[WAGON-474] Upgrade and revise all tests for Jetty 8







* Upgrade all test code to Jetty 8.1.22 and Servlet 3.0



* Unify variable names in redirect usecases to realServer and



  redirectServer



* RedirectHandler: redirect code is passed but completely ignored because



  sendRedirect() always sends 302



* Chronologically sort checkHandlerResult() calls



** Set redirect code (See Other (303)) as requested



** Update checkHandlerResult() for requested status codes rather sent



   chosen by server (mismatched previously)



* testPreemptiveAuthentication*(): properly check for OK for GET and



  CREATED for PUT instead of OK only for both



* WebDavWagonTest: replace status code literal for



  HttpServletResponse.SC_* for better readability



* testRedirect*(): add more checkHandlerResults











Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo



Commit:

http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e


Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e



Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e







Branch: refs/heads/jetty-8



Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee



Parents: b451e41



Author: Michael Osipov 



Authored: Mon Dec 26 00:55:09 2016 +0100



Committer: Michael Osipov 



Committed: Mon Dec 26 00:55:09 2016 +0100







--



 pom.xml |  14 +-



 wagon-provider-test/pom.xml |   8 +-



 .../maven/wagon/http/HttpWagonTestCase.java | 273

++-


 wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-



 .../http/LightweightHttpsWagonTest.java |   6 +-



 wagon-providers/wagon-http/pom.xml  |   8 +-



 .../http/HttpWagonHttpServerTestCase.java   |  14 +-



 .../http/HttpWagonReasonPhraseTest.java |   2 +-



 .../providers/http/HttpWagonTimeoutTest.java|   2 +-



 .../http/HttpsWagonPreemptiveTest.java  |   6 +-



 .../wagon/providers/http/HttpsWagonTest.java|   6 +-



 .../providers/http/HugeFileDownloadTest.java|  14 +-



 wagon-providers/wagon-ssh/pom.xml   |   8 +-



 .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-



 wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-



 .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---



 .../providers/webdav/WebDavsWagonTest.java  |   6 +-



 wagon-tcks/wagon-tck-http/pom.xml   |  12 +-



 .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--



 19 files changed, 278 insertions(+), 254 deletions(-)



--











http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml



--



diff --git a/pom.xml b/pom.xml



index e127da4..a90a1ba 100644



--- a/pom.xml



+++ b/pom.xml



@@ -313,14 +313,14 @@ under the License.



   







   



-org.mortbay.jetty



-jetty



-6.1.26



+org.eclipse.jetty.aggregate



+jetty-all



+8.1.22.v20160922



   



   



-org.mortbay.jetty



-servlet-api



-2.5-20081211



+javax.servlet



+javax.servlet-api



+3.0.1



 



 



   



@@ -565,5 +565,5 @@ under the License.



   



 



   



-



+



 







http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/



2e1c807e/wagon-provider-test/pom.xml



--



diff --git a/wagon-provider-test/pom.xml b/wagon-provider-test/pom.xml



index b19e7ac..9b0f4e5 100644



--- a/wagon-provider-test/pom.xml



+++ b/wagon-provider-test/pom.xml



@@ -51,12 +51,12 @@ under the License.



   compile



 



 



-  org.mortbay.jetty



-  jetty



+  org.eclipse.jetty.aggregate



+  jetty-all

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-28 Thread Olivier Lamy 
2017 is in few days so maybe java 1.7 shouldn't be a big problem

On Wed, 28 Dec 2016 at 11:09 am, Michael Osipov  wrote:

Am 2016-12-28 um 08:54 schrieb Olivier Lamy:

> Hi

> What about at least Jetty 9.2? (8.x is not anymore supported)



Wagon is still Java 1.6, therefore I had to stick to Jetty 8.

Moving from 8 to 9 is easier than from 6 to 9.



See:

https://www.eclipse.org/jetty/documentation/9.3.x/what-jetty-version.html



> On 26 December 2016 at 10:58,  wrote:

>

>> Repository: maven-wagon

>> Updated Branches:

>>   refs/heads/jetty-8 [created] e707a2691

>>

>>

>> [WAGON-474] Upgrade and revise all tests for Jetty 8

>>

>> * Upgrade all test code to Jetty 8.1.22 and Servlet 3.0

>> * Unify variable names in redirect usecases to realServer and

>>   redirectServer

>> * RedirectHandler: redirect code is passed but completely ignored because

>>   sendRedirect() always sends 302

>> * Chronologically sort checkHandlerResult() calls

>> ** Set redirect code (See Other (303)) as requested

>> ** Update checkHandlerResult() for requested status codes rather sent

>>chosen by server (mismatched previously)

>> * testPreemptiveAuthentication*(): properly check for OK for GET and

>>   CREATED for PUT instead of OK only for both

>> * WebDavWagonTest: replace status code literal for

>>   HttpServletResponse.SC_* for better readability

>> * testRedirect*(): add more checkHandlerResults

>>

>>

>> Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo

>> Commit:
http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e

>> Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e

>> Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e

>>

>> Branch: refs/heads/jetty-8

>> Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee

>> Parents: b451e41

>> Author: Michael Osipov 

>> Authored: Mon Dec 26 00:55:09 2016 +0100

>> Committer: Michael Osipov 

>> Committed: Mon Dec 26 00:55:09 2016 +0100

>>

>> --

>>  pom.xml |  14 +-

>>  wagon-provider-test/pom.xml |   8 +-

>>  .../maven/wagon/http/HttpWagonTestCase.java | 273
++-

>>  wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-

>>  .../http/LightweightHttpsWagonTest.java |   6 +-

>>  wagon-providers/wagon-http/pom.xml  |   8 +-

>>  .../http/HttpWagonHttpServerTestCase.java   |  14 +-

>>  .../http/HttpWagonReasonPhraseTest.java |   2 +-

>>  .../providers/http/HttpWagonTimeoutTest.java|   2 +-

>>  .../http/HttpsWagonPreemptiveTest.java  |   6 +-

>>  .../wagon/providers/http/HttpsWagonTest.java|   6 +-

>>  .../providers/http/HugeFileDownloadTest.java|  14 +-

>>  wagon-providers/wagon-ssh/pom.xml   |   8 +-

>>  .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-

>>  wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-

>>  .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---

>>  .../providers/webdav/WebDavsWagonTest.java  |   6 +-

>>  wagon-tcks/wagon-tck-http/pom.xml   |  12 +-

>>  .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--

>>  19 files changed, 278 insertions(+), 254 deletions(-)

>> --

>>

>>

>> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml

>> --

>> diff --git a/pom.xml b/pom.xml

>> index e127da4..a90a1ba 100644

>> --- a/pom.xml

>> +++ b/pom.xml

>> @@ -313,14 +313,14 @@ under the License.

>>

>>

>>

>> -org.mortbay.jetty

>> -jetty

>> -6.1.26

>> +org.eclipse.jetty.aggregate

>> +jetty-all

>> +8.1.22.v20160922

>>

>>

>> -org.mortbay.jetty

>> -servlet-api

>> -2.5-20081211

>> +javax.servlet

>> +javax.servlet-api

>> +3.0.1

>>  

>>  

>>

>> @@ -565,5 +565,5 @@ under the License.

>>

>>  

>>

>> -

>> +

>>  

>>

>> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/

>> 2e1c807e/wagon-provider-test/pom.xml

>> --

>> diff --git a/wagon-provider-test/pom.xml b/wagon-provider-test/pom.xml

>> index b19e7ac..9b0f4e5 100644

>> --- a/wagon-provider-test/pom.xml

>> +++ b/wagon-provider-test/pom.xml

>> @@ -51,12 +51,12 @@ under the License.

>>compile

>>  

>>  

>> -  org.mortbay.jetty

>> -  jetty

>> +  org.eclipse.jetty.aggregate

>> +  jetty-all

>>  

>>  

>> -  org.mortbay.jetty

>> -  servlet-api

>> +  javax.servlet

>> +  javax.servlet-api

>>  

>>  

>>

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-28 Thread Michael Osipov 

Am 2016-12-28 um 08:54 schrieb Olivier Lamy:

Hi
What about at least Jetty 9.2? (8.x is not anymore supported)


Wagon is still Java 1.6, therefore I had to stick to Jetty 8.
Moving from 8 to 9 is easier than from 6 to 9.

See: 
https://www.eclipse.org/jetty/documentation/9.3.x/what-jetty-version.html



On 26 December 2016 at 10:58,  wrote:


Repository: maven-wagon
Updated Branches:
  refs/heads/jetty-8 [created] e707a2691


[WAGON-474] Upgrade and revise all tests for Jetty 8

* Upgrade all test code to Jetty 8.1.22 and Servlet 3.0
* Unify variable names in redirect usecases to realServer and
  redirectServer
* RedirectHandler: redirect code is passed but completely ignored because
  sendRedirect() always sends 302
* Chronologically sort checkHandlerResult() calls
** Set redirect code (See Other (303)) as requested
** Update checkHandlerResult() for requested status codes rather sent
   chosen by server (mismatched previously)
* testPreemptiveAuthentication*(): properly check for OK for GET and
  CREATED for PUT instead of OK only for both
* WebDavWagonTest: replace status code literal for
  HttpServletResponse.SC_* for better readability
* testRedirect*(): add more checkHandlerResults


Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e
Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e
Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e

Branch: refs/heads/jetty-8
Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee
Parents: b451e41
Author: Michael Osipov 
Authored: Mon Dec 26 00:55:09 2016 +0100
Committer: Michael Osipov 
Committed: Mon Dec 26 00:55:09 2016 +0100

--
 pom.xml |  14 +-
 wagon-provider-test/pom.xml |   8 +-
 .../maven/wagon/http/HttpWagonTestCase.java | 273 ++-
 wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-
 .../http/LightweightHttpsWagonTest.java |   6 +-
 wagon-providers/wagon-http/pom.xml  |   8 +-
 .../http/HttpWagonHttpServerTestCase.java   |  14 +-
 .../http/HttpWagonReasonPhraseTest.java |   2 +-
 .../providers/http/HttpWagonTimeoutTest.java|   2 +-
 .../http/HttpsWagonPreemptiveTest.java  |   6 +-
 .../wagon/providers/http/HttpsWagonTest.java|   6 +-
 .../providers/http/HugeFileDownloadTest.java|  14 +-
 wagon-providers/wagon-ssh/pom.xml   |   8 +-
 .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-
 wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-
 .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---
 .../providers/webdav/WebDavsWagonTest.java  |   6 +-
 wagon-tcks/wagon-tck-http/pom.xml   |  12 +-
 .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--
 19 files changed, 278 insertions(+), 254 deletions(-)
--


http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml
--
diff --git a/pom.xml b/pom.xml
index e127da4..a90a1ba 100644
--- a/pom.xml
+++ b/pom.xml
@@ -313,14 +313,14 @@ under the License.
   

   
-org.mortbay.jetty
-jetty
-6.1.26
+org.eclipse.jetty.aggregate
+jetty-all
+8.1.22.v20160922
   
   
-org.mortbay.jetty
-servlet-api
-2.5-20081211
+javax.servlet
+javax.servlet-api
+3.0.1
 
 
   
@@ -565,5 +565,5 @@ under the License.
   
 
   
-
+
 

http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
2e1c807e/wagon-provider-test/pom.xml
--
diff --git a/wagon-provider-test/pom.xml b/wagon-provider-test/pom.xml
index b19e7ac..9b0f4e5 100644
--- a/wagon-provider-test/pom.xml
+++ b/wagon-provider-test/pom.xml
@@ -51,12 +51,12 @@ under the License.
   compile
 
 
-  org.mortbay.jetty
-  jetty
+  org.eclipse.jetty.aggregate
+  jetty-all
 
 
-  org.mortbay.jetty
-  servlet-api
+  javax.servlet
+  javax.servlet-api
 
 
   org.slf4j

http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
2e1c807e/wagon-provider-test/src/main/java/org/apache/maven/wagon/http/
HttpWagonTestCase.java
--
diff --git a/wagon-provider-test/src/main/java/org/apache/maven/
wagon/http/HttpWagonTestCase.java b/wagon-provider-test/src/
main/java/org/apache/maven/wagon/http/HttpWagonTestCase.java
index dfc499e..cf479fd 100644
--- a/wagon-provider-test/src/main/java/org/apache/maven/
wagon/http/HttpWagonTestCase.java
+++ b/wagon-provider-test/src/main/java/org/apache/maven/

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-27 Thread Olivier Lamy 
Hi
What about at least Jetty 9.2? (8.x is not anymore supported)


On 26 December 2016 at 10:58,  wrote:

> Repository: maven-wagon
> Updated Branches:
>   refs/heads/jetty-8 [created] e707a2691
>
>
> [WAGON-474] Upgrade and revise all tests for Jetty 8
>
> * Upgrade all test code to Jetty 8.1.22 and Servlet 3.0
> * Unify variable names in redirect usecases to realServer and
>   redirectServer
> * RedirectHandler: redirect code is passed but completely ignored because
>   sendRedirect() always sends 302
> * Chronologically sort checkHandlerResult() calls
> ** Set redirect code (See Other (303)) as requested
> ** Update checkHandlerResult() for requested status codes rather sent
>chosen by server (mismatched previously)
> * testPreemptiveAuthentication*(): properly check for OK for GET and
>   CREATED for PUT instead of OK only for both
> * WebDavWagonTest: replace status code literal for
>   HttpServletResponse.SC_* for better readability
> * testRedirect*(): add more checkHandlerResults
>
>
> Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e
> Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e
> Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e
>
> Branch: refs/heads/jetty-8
> Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee
> Parents: b451e41
> Author: Michael Osipov 
> Authored: Mon Dec 26 00:55:09 2016 +0100
> Committer: Michael Osipov 
> Committed: Mon Dec 26 00:55:09 2016 +0100
>
> --
>  pom.xml |  14 +-
>  wagon-provider-test/pom.xml |   8 +-
>  .../maven/wagon/http/HttpWagonTestCase.java | 273 ++-
>  wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-
>  .../http/LightweightHttpsWagonTest.java |   6 +-
>  wagon-providers/wagon-http/pom.xml  |   8 +-
>  .../http/HttpWagonHttpServerTestCase.java   |  14 +-
>  .../http/HttpWagonReasonPhraseTest.java |   2 +-
>  .../providers/http/HttpWagonTimeoutTest.java|   2 +-
>  .../http/HttpsWagonPreemptiveTest.java  |   6 +-
>  .../wagon/providers/http/HttpsWagonTest.java|   6 +-
>  .../providers/http/HugeFileDownloadTest.java|  14 +-
>  wagon-providers/wagon-ssh/pom.xml   |   8 +-
>  .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-
>  wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-
>  .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---
>  .../providers/webdav/WebDavsWagonTest.java  |   6 +-
>  wagon-tcks/wagon-tck-http/pom.xml   |  12 +-
>  .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--
>  19 files changed, 278 insertions(+), 254 deletions(-)
> --
>
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml
> --
> diff --git a/pom.xml b/pom.xml
> index e127da4..a90a1ba 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -313,14 +313,14 @@ under the License.
>
>
>
> -org.mortbay.jetty
> -jetty
> -6.1.26
> +org.eclipse.jetty.aggregate
> +jetty-all
> +8.1.22.v20160922
>
>
> -org.mortbay.jetty
> -servlet-api
> -2.5-20081211
> +javax.servlet
> +javax.servlet-api
> +3.0.1
>  
>  
>
> @@ -565,5 +565,5 @@ under the License.
>
>  
>
> -
> +
>  
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> 2e1c807e/wagon-provider-test/pom.xml
> --
> diff --git a/wagon-provider-test/pom.xml b/wagon-provider-test/pom.xml
> index b19e7ac..9b0f4e5 100644
> --- a/wagon-provider-test/pom.xml
> +++ b/wagon-provider-test/pom.xml
> @@ -51,12 +51,12 @@ under the License.
>compile
>  
>  
> -  org.mortbay.jetty
> -  jetty
> +  org.eclipse.jetty.aggregate
> +  jetty-all
>  
>  
> -  org.mortbay.jetty
> -  servlet-api
> +  javax.servlet
> +  javax.servlet-api
>  
>  
>org.slf4j
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> 2e1c807e/wagon-provider-test/src/main/java/org/apache/maven/wagon/http/
> HttpWagonTestCase.java
> --
> diff --git a/wagon-provider-test/src/main/java/org/apache/maven/
> wagon/http/HttpWagonTestCase.java b/wagon-provider-test/src/
> main/java/org/apache/maven/wagon/http/HttpWagonTestCase.java
> index dfc499e..cf479fd 100644
> --- a/wagon-provider-test/src/main/java/org/apache/maven/
> wagon/http/HttpWagonTestCase.java
> +++ b/wagon-provider-test/src/main/java/org/apache/maven/
>

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-27 Thread Michael Osipov 

Am 2016-12-26 um 02:56 schrieb Dan Tran:

redhat7/openjdk 7 ( latest)

this test fails on timeout.  I have seen this issue before not lately. Used
to work on  2.9, 2 10, and 2.11)


Please add the following to your Surefire config:

  
true
  
-MM-dd'T'HH:mm:ss.SSS
  
debug
  
error
  
debug
  
debug
  
debug
  
debug
  
debug
  
error


and we will see what the HttpClient does and why it is is failing. I ran 
preliminary tests on FC25 with OpenJDK 8, no issues so far.



-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-25 Thread Dan Tran 
redhat7/openjdk 7 ( latest)

this test fails on timeout.  I have seen this issue before not lately. Used
to work on  2.9, 2 10, and 2.11)

-D

On Sun, Dec 25, 2016 at 4:59 PM, Michael Osipov  wrote:

> Am 2016-12-26 um 01:45 schrieb Dan Tran:
>
>> this branch has one test consistently fail test on my local redhat 7,
>> java 7
>>
>> Running org.apache.maven.wagon.providers.http.HugeFileDownloadTest
>>
>
> What does it say? What JDK version do you use?
>
> This test works for me on two operating systems and five JDK versions.
>
>
> -
> To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
> For additional commands, e-mail: dev-h...@maven.apache.org
>
>

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-25 Thread Michael Osipov 

Am 2016-12-26 um 01:45 schrieb Dan Tran:

this branch has one test consistently fail test on my local redhat 7, java 7

Running org.apache.maven.wagon.providers.http.HugeFileDownloadTest


What does it say? What JDK version do you use?

This test works for me on two operating systems and five JDK versions.


-
To unsubscribe, e-mail: dev-unsubscr...@maven.apache.org
For additional commands, e-mail: dev-h...@maven.apache.org

Re: [1/2] maven-wagon git commit: [WAGON-474] Upgrade and revise all tests for Jetty 8

2016-12-25 Thread Dan Tran 
this branch has one test consistently fail test on my local redhat 7, java 7

Running org.apache.maven.wagon.providers.http.HugeFileDownloadTest




On Sun, Dec 25, 2016 at 3:58 PM,  wrote:

> Repository: maven-wagon
> Updated Branches:
>   refs/heads/jetty-8 [created] e707a2691
>
>
> [WAGON-474] Upgrade and revise all tests for Jetty 8
>
> * Upgrade all test code to Jetty 8.1.22 and Servlet 3.0
> * Unify variable names in redirect usecases to realServer and
>   redirectServer
> * RedirectHandler: redirect code is passed but completely ignored because
>   sendRedirect() always sends 302
> * Chronologically sort checkHandlerResult() calls
> ** Set redirect code (See Other (303)) as requested
> ** Update checkHandlerResult() for requested status codes rather sent
>chosen by server (mismatched previously)
> * testPreemptiveAuthentication*(): properly check for OK for GET and
>   CREATED for PUT instead of OK only for both
> * WebDavWagonTest: replace status code literal for
>   HttpServletResponse.SC_* for better readability
> * testRedirect*(): add more checkHandlerResults
>
>
> Project: http://git-wip-us.apache.org/repos/asf/maven-wagon/repo
> Commit: http://git-wip-us.apache.org/repos/asf/maven-wagon/commit/2e1c807e
> Tree: http://git-wip-us.apache.org/repos/asf/maven-wagon/tree/2e1c807e
> Diff: http://git-wip-us.apache.org/repos/asf/maven-wagon/diff/2e1c807e
>
> Branch: refs/heads/jetty-8
> Commit: 2e1c807e5345a1999376771f8b378b8cd3328fee
> Parents: b451e41
> Author: Michael Osipov 
> Authored: Mon Dec 26 00:55:09 2016 +0100
> Committer: Michael Osipov 
> Committed: Mon Dec 26 00:55:09 2016 +0100
>
> --
>  pom.xml |  14 +-
>  wagon-provider-test/pom.xml |   8 +-
>  .../maven/wagon/http/HttpWagonTestCase.java | 273 ++-
>  wagon-providers/wagon-http-lightweight/pom.xml  |   4 +-
>  .../http/LightweightHttpsWagonTest.java |   6 +-
>  wagon-providers/wagon-http/pom.xml  |   8 +-
>  .../http/HttpWagonHttpServerTestCase.java   |  14 +-
>  .../http/HttpWagonReasonPhraseTest.java |   2 +-
>  .../providers/http/HttpWagonTimeoutTest.java|   2 +-
>  .../http/HttpsWagonPreemptiveTest.java  |   6 +-
>  .../wagon/providers/http/HttpsWagonTest.java|   6 +-
>  .../providers/http/HugeFileDownloadTest.java|  14 +-
>  wagon-providers/wagon-ssh/pom.xml   |   8 +-
>  .../ssh/jsch/ScpWagonWithProxyTest.java |  16 +-
>  wagon-providers/wagon-webdav-jackrabbit/pom.xml |   8 +-
>  .../wagon/providers/webdav/WebDavWagonTest.java |  73 ++---
>  .../providers/webdav/WebDavsWagonTest.java  |   6 +-
>  wagon-tcks/wagon-tck-http/pom.xml   |  12 +-
>  .../wagon/tck/http/fixture/ServerFixture.java   |  52 ++--
>  19 files changed, 278 insertions(+), 254 deletions(-)
> --
>
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/2e1c807e/pom.xml
> --
> diff --git a/pom.xml b/pom.xml
> index e127da4..a90a1ba 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -313,14 +313,14 @@ under the License.
>
>
>
> -org.mortbay.jetty
> -jetty
> -6.1.26
> +org.eclipse.jetty.aggregate
> +jetty-all
> +8.1.22.v20160922
>
>
> -org.mortbay.jetty
> -servlet-api
> -2.5-20081211
> +javax.servlet
> +javax.servlet-api
> +3.0.1
>  
>  
>
> @@ -565,5 +565,5 @@ under the License.
>
>  
>
> -
> +
>  
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> 2e1c807e/wagon-provider-test/pom.xml
> --
> diff --git a/wagon-provider-test/pom.xml b/wagon-provider-test/pom.xml
> index b19e7ac..9b0f4e5 100644
> --- a/wagon-provider-test/pom.xml
> +++ b/wagon-provider-test/pom.xml
> @@ -51,12 +51,12 @@ under the License.
>compile
>  
>  
> -  org.mortbay.jetty
> -  jetty
> +  org.eclipse.jetty.aggregate
> +  jetty-all
>  
>  
> -  org.mortbay.jetty
> -  servlet-api
> +  javax.servlet
> +  javax.servlet-api
>  
>  
>org.slf4j
>
> http://git-wip-us.apache.org/repos/asf/maven-wagon/blob/
> 2e1c807e/wagon-provider-test/src/main/java/org/apache/maven/wagon/http/
> HttpWagonTestCase.java
> --
> diff --git a/wagon-provider-test/src/main/java/org/apache/maven/
> wagon/http/HttpWagonTestCase.java b/wagon-provider-test/src/
> main/java/org/apache/maven/wagon/http/HttpWagonTestCase.java
> index dfc499e..cf479fd 100644
> --- a/wagon-provider-test/src/main/java/org/apache/maven/
>