Re: HttpCore 5.4-alpha1 soon?

2025-07-16 Thread Arturo Bernal
Hi Oleg, +1 cheers, Arturo On Wed, Jul 16, 2025 at 2:42 PM Oleg Kalnichevski wrote: > Folks > > How do you feel about releasing HttpCore 5.4-alpha1 soon? > > Ryan, is there anything in core that you plan on implementing? Should > we upgrade HttpClient master to core 5.4 APIs? > > Oleg > > --

Re: HttpCore 5.4-alpha1 soon?

2025-07-16 Thread Gary Gregory
On Wed, Jul 16, 2025, 08:42 Oleg Kalnichevski wrote: > Folks > > How do you feel about releasing HttpCore 5.4-alpha1 soon? > Sounds good to me. > Ryan, is there anything in core that you plan on implementing? Should > we upgrade HttpClient master to core 5.4 APIs? > Sure, might as well iron o

HttpCore 5.4-alpha1 soon?

2025-07-16 Thread Oleg Kalnichevski
Folks How do you feel about releasing HttpCore 5.4-alpha1 soon?  Ryan, is there anything in core that you plan on implementing? Should we upgrade HttpClient master to core 5.4 APIs? Oleg - To unsubscribe, e-mail: dev-unsubscr..

[PR] HTTPCLIENT-2371: Improved request re-execution strategy [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c opened a new pull request, #679: URL: https://github.com/apache/httpcomponents-client/pull/679 @arturobernalg @michael-o @rschmitt @garydgregory Please do feel free to take a look at the proposed request re-execution API changes. Do feel free to propose better naming / better API ideas

Re: [PR] Deprecate setContentDecoderRegistry to populate contentDecoder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c commented on PR #675: URL: https://github.com/apache/httpcomponents-client/pull/675#issuecomment-3078382706 Superseded by #678 -- 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

Re: [PR] Deprecate setContentDecoderRegistry to populate contentDecoder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c closed pull request #675: Deprecate setContentDecoderRegistry to populate contentDecoder URL: https://github.com/apache/httpcomponents-client/pull/675 -- 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

Re: [PR] Undeprecate InputStreamFactory and restore contentDecoderMap functionality in HttpClientBuilder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c merged PR #678: URL: https://github.com/apache/httpcomponents-client/pull/678 -- 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...@hc.ap

Re: [PR] Deprecate setContentDecoderRegistry to populate contentDecoder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c commented on PR #675: URL: https://github.com/apache/httpcomponents-client/pull/675#issuecomment-3078274722 @arturobernalg I would like to propose an alternative approach. Could you live with #678? -- This is an automated message from the Apache Git Service. To respond to the messag

[PR] Undeprecate InputStreamFactory and restore contentDecoderMap functionality in HttpClientBuilder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c opened a new pull request, #678: URL: https://github.com/apache/httpcomponents-client/pull/678 @arturobernalg I realized you should not have deprecated `InputStreamFactory`. It was still perfectly useful at the HttpClientBuilder level for plugging in custom decoders. How about t

[jira] [Resolved] (HTTPCLIENT-2383) LaxRedirectStrategy unintentionally blocks redirects due to inherited sensitive header check

2025-07-16 Thread Arturo Bernal (Jira)
[ https://issues.apache.org/jira/browse/HTTPCLIENT-2383?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arturo Bernal resolved HTTPCLIENT-2383. --- Resolution: Fixed > LaxRedirectStrategy unintentionally blocks redirects due to

[jira] [Closed] (HTTPCLIENT-2383) LaxRedirectStrategy unintentionally blocks redirects due to inherited sensitive header check

2025-07-16 Thread Arturo Bernal (Jira)
[ https://issues.apache.org/jira/browse/HTTPCLIENT-2383?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Arturo Bernal closed HTTPCLIENT-2383. - > LaxRedirectStrategy unintentionally blocks redirects due to inherited > sensitive hea

Re: [PR] HTTPCLIENT-2383 Bypass sensitive-header check in LaxRedirectStrategy [httpcomponents-client]

2025-07-16 Thread via GitHub
arturobernalg merged PR #676: URL: https://github.com/apache/httpcomponents-client/pull/676 -- 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

[jira] [Commented] (HTTPCLIENT-2383) LaxRedirectStrategy unintentionally blocks redirects due to inherited sensitive header check

2025-07-16 Thread ASF subversion and git services (Jira)
[ https://issues.apache.org/jira/browse/HTTPCLIENT-2383?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18007555#comment-18007555 ] ASF subversion and git services commented on HTTPCLIENT-2383:

Re: [PR] [HTTPCLIENT-2383] Make LaxRedirectStrategy to allow redirects if sensitive headers are part of the request [httpcomponents-client]

2025-07-16 Thread via GitHub
arturobernalg closed pull request #677: [HTTPCLIENT-2383] Make LaxRedirectStrategy to allow redirects if sensitive headers are part of the request URL: https://github.com/apache/httpcomponents-client/pull/677 -- This is an automated message from the Apache Git Service. To respond to the mess

Re: [PR] Deprecate setContentDecoderRegistry to populate contentDecoder [httpcomponents-client]

2025-07-16 Thread via GitHub
arturobernalg commented on PR #675: URL: https://github.com/apache/httpcomponents-client/pull/675#issuecomment-3078121854 > @arturobernalg I spotted another problem. The `LinkedHashMap>` map used by `HttpClientBuilder` basically means one cannot plug-in 3rd party coders if they are not in

Re: [PR] HTTPCLIENT-2383 Bypass sensitive-header check in LaxRedirectStrategy [httpcomponents-client]

2025-07-16 Thread via GitHub
arturobernalg commented on PR #676: URL: https://github.com/apache/httpcomponents-client/pull/676#issuecomment-3078039433 > Included some more tests here: #677 that could be interesting for you @dani0600 cherry picked [6fcd35c](https://github.com/apache/httpcomponents-client/pull/676

Re: [PR] [HTTPCLIENT-2383] Make LaxRedirectStrategy to allow redirects if sensitive headers are part of the request [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c commented on code in PR #677: URL: https://github.com/apache/httpcomponents-client/pull/677#discussion_r2209974667 ## httpclient5/src/test/java/org/apache/hc/client5/http/impl/TestDefaultRedirectStrategy.java: ## @@ -261,13 +261,6 @@ void testRedirectAllowed() throws Excep

Re: [PR] [HTTPCLIENT-2383] Make LaxRedirectStrategy to allow redirects if sensitive headers are part of the request [httpcomponents-client]

2025-07-16 Thread via GitHub
dani0600 commented on code in PR #677: URL: https://github.com/apache/httpcomponents-client/pull/677#discussion_r2209966110 ## httpclient5/src/test/java/org/apache/hc/client5/http/impl/TestDefaultRedirectStrategy.java: ## @@ -261,13 +261,6 @@ void testRedirectAllowed() throws E

Re: [PR] [HTTPCLIENT-2383] Make LaxRedirectStrategy to allow redirects if sensitive headers are part of the request [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c commented on code in PR #677: URL: https://github.com/apache/httpcomponents-client/pull/677#discussion_r2209957853 ## httpclient5/src/test/java/org/apache/hc/client5/http/impl/TestDefaultRedirectStrategy.java: ## @@ -261,13 +261,6 @@ void testRedirectAllowed() throws Excep

Re: [PR] Deprecate setContentDecoderRegistry to populate contentDecoder [httpcomponents-client]

2025-07-16 Thread via GitHub
ok2c commented on PR #675: URL: https://github.com/apache/httpcomponents-client/pull/675#issuecomment-3077967310 @arturobernalg I spotted another problem. The `LinkedHashMap>` map used by `HttpClientBuilder` basically means one cannot plug-in 3rd party coders if they are not in present in `

[jira] [Commented] (HTTPCLIENT-2382) Apache HttpClient removes the `Content-Length` header from a DELETE request even when the request has a body and the header is explicitly set.

2025-07-16 Thread Anas Quazbary (Jira)
[ https://issues.apache.org/jira/browse/HTTPCLIENT-2382?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=18007540#comment-18007540 ] Anas Quazbary commented on HTTPCLIENT-2382: --- i have succeeded to send a DE

[jira] [Closed] (HTTPCLIENT-2382) Apache HttpClient removes the `Content-Length` header from a DELETE request even when the request has a body and the header is explicitly set.

2025-07-16 Thread Anas Quazbary (Jira)
[ https://issues.apache.org/jira/browse/HTTPCLIENT-2382?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Anas Quazbary closed HTTPCLIENT-2382. - Fix Version/s: (was: 5.3.1) Resolution: Not A Bug > Apache HttpClient rem