[GitHub] [cxf] rmannibucau commented on issue #660: [CXF-8263] Support SSEs in MP Rest Client 2.0

2020-04-20 Thread GitBox
rmannibucau commented on issue #660: URL: https://github.com/apache/cxf/pull/660#issuecomment-616679437 @reta hmm, MP reactive-streams-operators was supposed to redefine (with the same pitfall that it is no more aligned on what people use on the JVM). Seems it hasnt be done but the spec

[GitHub] [cxf] reta commented on issue #660: [CXF-8263] Support SSEs in MP Rest Client 2.0

2020-04-20 Thread GitBox
reta commented on issue #660: URL: https://github.com/apache/cxf/pull/660#issuecomment-616724202 @rmannibucau It certainly makes sense (implementation wise the PR needs some work, the ForkJoin pool was also on my list but I thought to have TCK tests first). Yeah, you seem to have more

Segregate client and server logging

2020-04-20 Thread Mohana Rao S V
In the existing `org.apache.cxf.ext.logging.LoggingFeature` implementation it is possible to segregate the server jaxws/jaxrs logs and It would be nice to enhance to feature to segregate server/client logs. Ex: Ideally application can have jaxrs Rest API at the same time it can have rest-client

[GitHub] [cxf] coheigea commented on a change in pull request #638: upgrade guava, CVE-2018-10237

2020-04-20 Thread GitBox
coheigea commented on a change in pull request #638: URL: https://github.com/apache/cxf/pull/638#discussion_r377614476 ## File path: parent/pom.xml ## @@ -119,7 +119,7 @@ 3.1.4 1.0.12 1.0.4 -20.0 +28.2-jre Review comment:

[GitHub] [cxf] coheigea commented on a change in pull request #638: upgrade guava, CVE-2018-10237

2020-04-20 Thread GitBox
coheigea commented on a change in pull request #638: URL: https://github.com/apache/cxf/pull/638#discussion_r411227776 ## File path: parent/pom.xml ## @@ -119,7 +119,7 @@ 3.1.4 1.0.12 1.0.4 -20.0 +28.2-jre Review comment:

[GitHub] [cxf] reta commented on a change in pull request #638: upgrade guava, CVE-2018-10237

2020-04-20 Thread GitBox
reta commented on a change in pull request #638: URL: https://github.com/apache/cxf/pull/638#discussion_r411340344 ## File path: parent/pom.xml ## @@ -119,7 +119,7 @@ 3.1.4 1.0.12 1.0.4 -20.0 +28.2-jre Review comment: :+1:

[GitHub] [cxf] coheigea commented on a change in pull request #638: upgrade guava, CVE-2018-10237

2020-04-20 Thread GitBox
coheigea commented on a change in pull request #638: URL: https://github.com/apache/cxf/pull/638#discussion_r411351629 ## File path: parent/pom.xml ## @@ -119,7 +119,7 @@ 3.1.4 1.0.12 1.0.4 -20.0 +28.2-jre Review comment: +1

[GitHub] [cxf] andrei-ivanov commented on a change in pull request #638: upgrade guava, CVE-2018-10237

2020-04-20 Thread GitBox
andrei-ivanov commented on a change in pull request #638: URL: https://github.com/apache/cxf/pull/638#discussion_r411349925 ## File path: parent/pom.xml ## @@ -119,7 +119,7 @@ 3.1.4 1.0.12 1.0.4 -20.0 +28.2-jre Review comment:

[GitHub] [cxf] reta commented on issue #660: [CXF-8263] Support SSEs in MP Rest Client 2.0

2020-04-20 Thread GitBox
reta commented on issue #660: URL: https://github.com/apache/cxf/pull/660#issuecomment-616577276 @rmannibucau I found Reactive Streams to be good abstraction, every single reactive stream implementation supports it (so interop would be the least concern here I believe), afaik even MP does