rzo1 opened a new pull request, #1822:
URL: https://github.com/apache/cxf/pull/1822
Hi all,
I am currently checking the integration of CXF 4.1.0-SNAPSHOT into TomEE 10
and I noticed a change in `AbstractHTTPDestination`.
An additional `try-catch` was added around `getUserPrinci
rzo1 commented on PR #1822:
URL: https://github.com/apache/cxf/pull/1822#issuecomment-2066319454
Ok from what I can read in the commit history, it seems, it was introduced
because of jetty 12?
The underlying exception (reason for failing GH actions) is a NPE in Jetty:
```
[I
reta commented on PR #1777:
URL: https://github.com/apache/cxf/pull/1777#issuecomment-2066394896
Thanks a lot @jimma !
> For the httpclient executor, should it set to some shared new fixed size
thread pool like cpu processor count *2 by default
> instead of using JDK's `Executors.n
reta merged PR #1820:
URL: https://github.com/apache/cxf/pull/1820
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org
For q
reta merged PR #1821:
URL: https://github.com/apache/cxf/pull/1821
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org
For q
ffang commented on PR #1822:
URL: https://github.com/apache/cxf/pull/1822#issuecomment-2066538612
Hi @rzo1 ,
Yes, this change is caused by Jetty12 upgrade. And it's good that we just
catch NPE from jetty other than more generic exceptions. Thanks for reporting
and fixing this!
reta commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572368151
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchange
reta merged PR #1777:
URL: https://github.com/apache/cxf/pull/1777
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org
For q
rzo1 commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572491679
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchange
ffang commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572581658
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchang
dependabot[bot] commented on PR #1816:
URL: https://github.com/apache/cxf/pull/1816#issuecomment-2066966329
OK, I won't notify you again about this release, but will get in touch when
a new version is available. If you'd rather skip all updates until the next
major or minor version, let me
reta closed pull request #1816: Bump
jakarta.enterprise:jakarta.enterprise.cdi-api from 4.0.1 to 4.1.0
URL: https://github.com/apache/cxf/pull/1816
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to th
reta commented on code in PR #1819:
URL: https://github.com/apache/cxf/pull/1819#discussion_r1572837369
##
core/src/main/java/org/apache/cxf/transport/common/gzip/GZIPOutInterceptor.java:
##
@@ -337,7 +337,10 @@ private static void addHeader(Message message, String
name, String
ffang commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572882731
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchang
rzo1 commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572898811
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchange
rzo1 closed pull request #1822: Exception for getUserPrincipal() in
AbstractHTTPDestination is slurped since CXF 4.1.x
URL: https://github.com/apache/cxf/pull/1822
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL a
ffang merged PR #1823:
URL: https://github.com/apache/cxf/pull/1823
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: dev-unsubscr...@cxf.apache.org
For
ffang commented on PR #1823:
URL: https://github.com/apache/cxf/pull/1823#issuecomment-2067229753
Thanks @reta !
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscr
ffang commented on code in PR #1822:
URL: https://github.com/apache/cxf/pull/1822#discussion_r1572923309
##
rt/transports/http/src/main/java/org/apache/cxf/transport/http/AbstractHTTPDestination.java:
##
@@ -410,8 +410,12 @@ private boolean isWSAddressingReplyToSpecified(Exchang
19 matches
Mail list logo