Re: [PR] HTTPCLIENT-2301. Fix stale connection handling in BasicHttpClientConnectionManager [httpcomponents-client]

2023-11-05 Thread via GitHub
ok2c commented on code in PR #502: URL: https://github.com/apache/httpcomponents-client/pull/502#discussion_r1382557118 ## httpclient5/src/main/java/org/apache/hc/client5/http/impl/io/BasicHttpClientConnectionManager.java: ## @@ -357,7 +357,19 @@ ManagedHttpClientConnection

[PR] HTTPCORE-761: support ExtendedSocketOption in socket [httpcomponents-core]

2023-11-05 Thread via GitHub
kkewwei opened a new pull request, #442: URL: https://github.com/apache/httpcomponents-core/pull/442 Support setting TCP_KEEPIDLE, TCP_KEEPINTERVAL, TCP_KEEPCOUNT in Socket. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] HTTPCORE-761: support ExtendedSocketOption in socket [httpcomponents-core]

2023-11-05 Thread via GitHub
ok2c commented on code in PR #442: URL: https://github.com/apache/httpcomponents-core/pull/442#discussion_r1382621660 ## httpcore5/src/main/java/org/apache/hc/core5/reactor/SingleCoreIOReactor.java: ## @@ -27,10 +27,22 @@ package org.apache.hc.core5.reactor; +import

Re: [PR] HTTPCLIENT-2277: RFC 9111 conformance improvements / bug fixes [httpcomponents-client]

2023-11-05 Thread via GitHub
ok2c commented on PR #501: URL: https://github.com/apache/httpcomponents-client/pull/501#issuecomment-1793709117 @arturobernalg Please review. -- 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

Re: [PR] HTTPCLIENT-2301. Fix stale connection handling in BasicHttpClientConnectionManager [httpcomponents-client]

2023-11-05 Thread via GitHub
arturobernalg commented on code in PR #502: URL: https://github.com/apache/httpcomponents-client/pull/502#discussion_r1382565133 ## httpclient5/src/main/java/org/apache/hc/client5/http/impl/io/BasicHttpClientConnectionManager.java: ## @@ -357,7 +357,19 @@

Re: [PR] HTTPCORE-761: support ExtendedSocketOption in socket [httpcomponents-core]

2023-11-05 Thread via GitHub
ok2c commented on code in PR #442: URL: https://github.com/apache/httpcomponents-core/pull/442#discussion_r1382594611 ## httpcore5/src/main/java/org/apache/hc/core5/net/ExtendedSocketOptions.java: ## @@ -0,0 +1,49 @@ +/* + *

Re: [PR] HTTPCLIENT-2301. Fix stale connection handling in BasicHttpClientConnectionManager [httpcomponents-client]

2023-11-07 Thread via GitHub
arturobernalg commented on code in PR #502: URL: https://github.com/apache/httpcomponents-client/pull/502#discussion_r1385636960 ## httpclient5/src/main/java/org/apache/hc/client5/http/impl/io/BasicHttpClientConnectionManager.java: ## @@ -357,7 +357,19 @@

[PR] Add Zstd support to ContentCompressionExec. [httpcomponents-client]

2023-10-31 Thread via GitHub
arturobernalg opened a new pull request, #498: URL: https://github.com/apache/httpcomponents-client/pull/498 This update allows automatic selection of Zstd as a compression algorithm when available. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Add Zstd support to ContentCompressionExec. [httpcomponents-client]

2023-10-31 Thread via GitHub
garydgregory commented on PR #498: URL: https://github.com/apache/httpcomponents-client/pull/498#issuecomment-1786950363 Why not do this through Commons Compress? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

Re: [PR] Add Zstd support to ContentCompressionExec. [httpcomponents-client]

2023-10-31 Thread via GitHub
arturobernalg commented on PR #498: URL: https://github.com/apache/httpcomponents-client/pull/498#issuecomment-1787585157 > Why not do this through Commons Compress? In general keep it simple. Adding a direct dependency on the Zstd library is a straightforward way to enable

Re: [PR] Refactor ExponentialBackoffManager Tests to Remove Thread.sleep(). [httpcomponents-client]

2023-10-23 Thread via GitHub
arturobernalg commented on PR #495: URL: https://github.com/apache/httpcomponents-client/pull/495#issuecomment-1775899664 > TestLinearBackoffManager @ok2c Was modify a couple day before

Re: [PR] Refactor ExponentialBackoffManager Tests to Remove Thread.sleep(). [httpcomponents-client]

2023-10-24 Thread via GitHub
arturobernalg merged PR #495: URL: https://github.com/apache/httpcomponents-client/pull/495 -- 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:

Re: [PR] Refactor RequestIfRange class to use DateUtils for date parsing. [httpcomponents-client]

2023-10-24 Thread via GitHub
arturobernalg commented on code in PR #497: URL: https://github.com/apache/httpcomponents-client/pull/497#discussion_r1370687082 ## httpclient5/src/main/java/org/apache/hc/client5/http/protocol/RequestIfRange.java: ## @@ -121,36 +113,21 @@ public void process(final HttpRequest

Re: [PR] Refactor RequestIfRange class to use DateUtils for date parsing. [httpcomponents-client]

2023-10-24 Thread via GitHub
arturobernalg merged PR #497: URL: https://github.com/apache/httpcomponents-client/pull/497 -- 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:

Re: [PR] HTTPCORE-761: support ExtendedSocketOption in socket [httpcomponents-core]

2023-11-05 Thread via GitHub
kkewwei commented on code in PR #442: URL: https://github.com/apache/httpcomponents-core/pull/442#discussion_r1382699123 ## httpcore5/src/main/java/org/apache/hc/core5/reactor/SingleCoreIOReactor.java: ## @@ -27,10 +27,22 @@ package org.apache.hc.core5.reactor; +import

Re: [PR] HTTPCORE-761: support ExtendedSocketOption in socket [httpcomponents-core]

2023-11-06 Thread via GitHub
ok2c merged PR #442: URL: https://github.com/apache/httpcomponents-core/pull/442 -- 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:

[PR] HTTPCLIENT-2277: RFC 9111 conformance improvements / bug fixes [httpcomponents-client]

2023-11-04 Thread via GitHub
ok2c opened a new pull request, #501: URL: https://github.com/apache/httpcomponents-client/pull/501 * Aligns the behavior of the caching protocol handlers with the specification requirements (sections 3 and 4 RFC 9111) * Bug fixes -- This is an automated message from the Apache Git

Re: [PR] HTTPCLIENT-2277: RFC 9111 conformance improvements / bug fixes [httpcomponents-client]

2023-11-04 Thread via GitHub
ok2c commented on PR #501: URL: https://github.com/apache/httpcomponents-client/pull/501#issuecomment-1793404009 @arturobernalg This PR is still work in progress. Do take a look at the changes in `ResponseCachingPolicy` and `CachedResponseSuitabilityChecker` and let me know if find

[GitHub] [httpcomponents-core] arturobernalg commented on pull request #430: HTTPCORE-756 - Implement 'If-Range' request validation as per RFC 9110

2023-09-19 Thread via GitHub
arturobernalg commented on PR #430: URL: https://github.com/apache/httpcomponents-core/pull/430#issuecomment-1726330738 > * This change-set is definitely useful, but it kind of goes counter to the overall philosophy of the library. Generally we provide higher level APIs for consumers to

[GitHub] [httpcomponents-core] arturobernalg closed pull request #430: HTTPCORE-756 - Implement 'If-Range' request validation as per RFC 9110

2023-09-19 Thread via GitHub
arturobernalg closed pull request #430: HTTPCORE-756 - Implement 'If-Range' request validation as per RFC 9110 URL: https://github.com/apache/httpcomponents-core/pull/430 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #428: Add validation for :path pseudo-header in HTTP/2 requests RFC7540

2023-09-19 Thread via GitHub
arturobernalg commented on code in PR #428: URL: https://github.com/apache/httpcomponents-core/pull/428#discussion_r1330579222 ## httpcore5-h2/src/main/java/org/apache/hc/core5/http2/impl/DefaultH2RequestConverter.java: ## @@ -208,4 +210,39 @@ public List convert(final

[GitHub] [httpcomponents-client] arturobernalg commented on a diff in pull request #484: HTTPCLIENT-2277: deprecation of obsolete config parameters and removal of oudated or meaningless tests

2023-09-19 Thread via GitHub
arturobernalg commented on code in PR #484: URL: https://github.com/apache/httpcomponents-client/pull/484#discussion_r1330608339 ## httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/CacheConfig.java: ## @@ -257,11 +232,11 @@ public int

[GitHub] [httpcomponents-client] ok2c commented on a diff in pull request #484: HTTPCLIENT-2277: deprecation of obsolete config parameters and removal of oudated or meaningless tests

2023-09-19 Thread via GitHub
ok2c commented on code in PR #484: URL: https://github.com/apache/httpcomponents-client/pull/484#discussion_r1330654678 ## httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/CacheConfig.java: ## @@ -257,11 +232,11 @@ public int getMaxUpdateRetries(){ }

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #431: HTTP message support improvements

2023-09-22 Thread via GitHub
arturobernalg commented on code in PR #431: URL: https://github.com/apache/httpcomponents-core/pull/431#discussion_r1334759075 ## httpcore5/src/main/java/org/apache/hc/core5/http/impl/io/DefaultBHttpClientConnection.java: ## @@ -271,7 +271,7 @@ public void

[GitHub] [httpcomponents-client] arturobernalg merged pull request #486: HTTPCLIENT-2293 - Implement RFC-compliant TRACE request interceptor

2023-09-23 Thread via GitHub
arturobernalg merged PR #486: URL: https://github.com/apache/httpcomponents-client/pull/486 -- 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:

[GitHub] [httpcomponents-client] arturobernalg merged pull request #485: HTTPCLIENT-2293 - Implement 'If-Range' request validation as per RFC9110

2023-09-23 Thread via GitHub
arturobernalg merged PR #485: URL: https://github.com/apache/httpcomponents-client/pull/485 -- 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:

[GitHub] [httpcomponents-core] ok2c merged pull request #431: HTTP message support improvements

2023-09-23 Thread via GitHub
ok2c merged PR #431: URL: https://github.com/apache/httpcomponents-core/pull/431 -- 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:

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #431: HTTP message support improvements

2023-09-23 Thread via GitHub
ok2c commented on code in PR #431: URL: https://github.com/apache/httpcomponents-core/pull/431#discussion_r1334945292 ## httpcore5/src/main/java/org/apache/hc/core5/http/impl/DefaultConnectionReuseStrategy.java: ## @@ -81,7 +81,7 @@ public boolean keepAlive(

[GitHub] [httpcomponents-client] arturobernalg commented on a diff in pull request #486: HTTPCLIENT-2293 - Implement RFC-compliant TRACE request interceptor

2023-09-22 Thread via GitHub
arturobernalg commented on code in PR #486: URL: https://github.com/apache/httpcomponents-client/pull/486#discussion_r1334628672 ## httpclient5/src/main/java/org/apache/hc/client5/http/protocol/RequestTraceInterceptor.java: ## @@ -0,0 +1,143 @@ +/* + *

[GitHub] [httpcomponents-client] arturobernalg merged pull request #484: HTTPCLIENT-2277: deprecation of obsolete config parameters and removal of oudated or meaningless tests

2023-09-21 Thread via GitHub
arturobernalg merged PR #484: URL: https://github.com/apache/httpcomponents-client/pull/484 -- 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:

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #428: Add validation for :path pseudo-header in HTTP/2 requests RFC7540

2023-09-18 Thread via GitHub
ok2c commented on code in PR #428: URL: https://github.com/apache/httpcomponents-core/pull/428#discussion_r1329232659 ## httpcore5-h2/src/main/java/org/apache/hc/core5/http2/impl/DefaultH2RequestConverter.java: ## @@ -208,4 +210,39 @@ public List convert(final HttpRequest

[GitHub] [httpcomponents-core] arturobernalg commented on pull request #428: Add validation for :path pseudo-header in HTTP/2 requests RFC7540

2023-09-18 Thread via GitHub
arturobernalg commented on PR #428: URL: https://github.com/apache/httpcomponents-core/pull/428#issuecomment-1724146949 > @arturobernalg What RFC have you been using as a reference for change? @ok2c this one

[GitHub] [httpcomponents-core] ok2c commented on pull request #431: HTTP message support improvements

2023-09-22 Thread via GitHub
ok2c commented on PR #431: URL: https://github.com/apache/httpcomponents-core/pull/431#issuecomment-1730944357 @arturobernalg Please kindly let me know if you see anything wrong. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [httpcomponents-core] ok2c opened a new pull request, #431: HTTP message support improvements

2023-09-22 Thread via GitHub
ok2c opened a new pull request, #431: URL: https://github.com/apache/httpcomponents-core/pull/431 This patch corrects the name/value parsing code in `BasicHeaderValueParser` and adds a number of message element formatting and parsing methods to `MessageSupport` -- This is an automated

[GitHub] [httpcomponents-client] arturobernalg commented on a diff in pull request #484: HTTPCLIENT-2277: deprecation of obsolete config parameters and removal of oudated or meaningless tests

2023-09-20 Thread via GitHub
arturobernalg commented on code in PR #484: URL: https://github.com/apache/httpcomponents-client/pull/484#discussion_r1331915022 ## httpclient5-cache/src/main/java/org/apache/hc/client5/http/impl/cache/CacheConfig.java: ## @@ -257,11 +232,11 @@ public int

[GitHub] [httpcomponents-core] arturobernalg merged pull request #428: Add validation for :path pseudo-header in HTTP/2 requests RFC7540

2023-09-20 Thread via GitHub
arturobernalg merged PR #428: URL: https://github.com/apache/httpcomponents-core/pull/428 -- 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:

[GitHub] [httpcomponents-core] ok2c opened a new pull request, #434: HTTPCORE-756: server-side request / response protocol conformance checks

2023-09-28 Thread via GitHub
ok2c opened a new pull request, #434: URL: https://github.com/apache/httpcomponents-core/pull/434 * Moved outgoing response conformance checks to the correct place in the protocol pipeline * Added incoming request conformance checks based on RFC 9110. -- This is an automated message

[GitHub] [httpcomponents-core] ok2c commented on pull request #434: HTTPCORE-756: server-side request / response protocol conformance checks

2023-09-28 Thread via GitHub
ok2c commented on PR #434: URL: https://github.com/apache/httpcomponents-core/pull/434#issuecomment-1739333827 @arturobernalg Please take a look. -- 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

[GitHub] [httpcomponents-client] arturobernalg closed pull request #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status

2023-09-28 Thread via GitHub
arturobernalg closed pull request #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status URL: https://github.com/apache/httpcomponents-client/pull/489 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [httpcomponents-client] arturobernalg commented on pull request #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status

2023-09-28 Thread via GitHub
arturobernalg commented on PR #489: URL: https://github.com/apache/httpcomponents-client/pull/489#issuecomment-1739847709 > @arturobernalg The decision whether or not a request should be retried based on the response status is delegated to the `HttpRequestRetryStrategy`. The default

[GitHub] [httpcomponents-client] snehanie opened a new pull request, #488: Update MemcachedHttpCacheStorage.java

2023-09-25 Thread via GitHub
snehanie opened a new pull request, #488: URL: https://github.com/apache/httpcomponents-client/pull/488 UpdateCAS does not catch timeout exceptions. getUpdateCAS catches and throws ResourceIOException, but updateCas does not. Try and catch is important since BasicHttpCache would

[GitHub] [httpcomponents-core] chrysophylax opened a new pull request, #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
chrysophylax opened a new pull request, #435: URL: https://github.com/apache/httpcomponents-core/pull/435 Quick fix for maximum compatibility per JIRA -- 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

[GitHub] [httpcomponents-core] reschke commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
reschke commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341262963 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request,

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341269898 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request, final

[GitHub] [httpcomponents-core] reschke commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
reschke commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341287246 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request,

[GitHub] [httpcomponents-core] dependabot[bot] closed pull request #420: Bump io.reactivex.rxjava3:rxjava from 3.1.6 to 3.1.7

2023-09-29 Thread via GitHub
dependabot[bot] closed pull request #420: Bump io.reactivex.rxjava3:rxjava from 3.1.6 to 3.1.7 URL: https://github.com/apache/httpcomponents-core/pull/420 -- 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

[GitHub] [httpcomponents-core] dependabot[bot] opened a new pull request, #436: Bump io.reactivex.rxjava3:rxjava from 3.1.6 to 3.1.8

2023-09-29 Thread via GitHub
dependabot[bot] opened a new pull request, #436: URL: https://github.com/apache/httpcomponents-core/pull/436 Bumps [io.reactivex.rxjava3:rxjava](https://github.com/ReactiveX/RxJava) from 3.1.6 to 3.1.8. Release notes Sourced from

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341257383 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request, final

[GitHub] [httpcomponents-core] ok2c commented on pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c commented on PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#issuecomment-1740750258 @chrysophylax Please also make sure the build still passes all style checks. I would also kindly ask you to add a test case to `TestStandardInterceptors` for the new behavior.

[GitHub] [httpcomponents-core] chrysophylax commented on pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
chrysophylax commented on PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#issuecomment-1740848880 I updated the existing `testRequestContentNullEntity` test to the new expected behaviour and added a test to make sure non-PUT/POST/PATCH don't get the `Content-Length`

[GitHub] [httpcomponents-core] ok2c merged pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c merged PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435 -- 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:

[GitHub] [httpcomponents-client] ok2c closed pull request #488: Update MemcachedHttpCacheStorage.java

2023-09-29 Thread via GitHub
ok2c closed pull request #488: Update MemcachedHttpCacheStorage.java URL: https://github.com/apache/httpcomponents-client/pull/488 -- 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

[GitHub] [httpcomponents-client] ok2c commented on pull request #488: Update MemcachedHttpCacheStorage.java

2023-09-29 Thread via GitHub
ok2c commented on PR #488: URL: https://github.com/apache/httpcomponents-client/pull/488#issuecomment-1740897624 Superseded by 19abc7e947a25afcfa84f1efd4a915e7a7e8 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [httpcomponents-core] reschke commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
reschke commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341273913 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request,

[GitHub] [httpcomponents-core] chrysophylax commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
chrysophylax commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341277979 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341330485 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request, final

[GitHub] [httpcomponents-core] chrysophylax commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
chrysophylax commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341265651 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest

[GitHub] [httpcomponents-core] ok2c commented on a diff in pull request #435: HTTPCORE-759 Content-Length missing on null entity request content

2023-09-29 Thread via GitHub
ok2c commented on code in PR #435: URL: https://github.com/apache/httpcomponents-core/pull/435#discussion_r1341285734 ## httpcore5/src/main/java/org/apache/hc/core5/http/protocol/RequestContent.java: ## @@ -110,6 +110,10 @@ public void process(final HttpRequest request, final

[GitHub] [httpcomponents-core] dependabot[bot] commented on pull request #420: Bump io.reactivex.rxjava3:rxjava from 3.1.6 to 3.1.7

2023-09-29 Thread via GitHub
dependabot[bot] commented on PR #420: URL: https://github.com/apache/httpcomponents-core/pull/420#issuecomment-1740998514 Superseded by #436. -- 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

[GitHub] [httpcomponents-client] snehanie commented on pull request #488: Update MemcachedHttpCacheStorage.java

2023-09-29 Thread via GitHub
snehanie commented on PR #488: URL: https://github.com/apache/httpcomponents-client/pull/488#issuecomment-1741122926 Thank you for the fix. I was going to raise another pull request and you beat me to it. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [httpcomponents-client] ok2c commented on pull request #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status

2023-09-28 Thread via GitHub
ok2c commented on PR #489: URL: https://github.com/apache/httpcomponents-client/pull/489#issuecomment-1738654815 @arturobernalg The decision whether or not a request should be retried based on the response status is delegated to the `HttpRequestRetryStrategy`. The default implementation

[GitHub] [httpcomponents-core] arturobernalg merged pull request #434: HTTPCORE-756: server-side request / response protocol conformance checks

2023-09-30 Thread via GitHub
arturobernalg merged PR #434: URL: https://github.com/apache/httpcomponents-core/pull/434 -- 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:

[GitHub] [httpcomponents-core] ok2c opened a new pull request, #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
ok2c opened a new pull request, #437: URL: https://github.com/apache/httpcomponents-core/pull/437 (no comment) -- 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

[GitHub] [httpcomponents-core] ok2c commented on pull request #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
ok2c commented on PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#issuecomment-1742080946 @arturobernalg Please take a look. -- 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

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
Marcono1234 commented on code in PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#discussion_r1343057798 ## httpclient5/src/main/java/org/apache/hc/client5/http/ssl/TrustAllStrategy.java: ## @@ -34,9 +34,15 @@ import

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
Marcono1234 commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1343067206 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1343130372 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1343130372 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1343130372 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
ok2c commented on PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#issuecomment-1743706149 @Marcono1234 Just deprecate `TrustSelfSignedStrategy`in the same PR. I would not bother with a backport of the changes to 4.x but I will merge them if you raise a PR for the

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
Marcono1234 commented on PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#issuecomment-1743608258 > I also propose TrustSelfSignedStrategy be deprecated in favor of SSLContextBuilder. The class is basically useless. Should I include this in this PR or create a

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#discussion_r1343119220 ## httpclient5/src/main/java/org/apache/hc/client5/http/ssl/TrustAllStrategy.java: ## @@ -34,9 +34,15 @@ import org.apache.hc.core5.ssl.TrustStrategy;

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
arturobernalg commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342183980 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override

[GitHub] [httpcomponents-client] Marcono1234 opened a new pull request, #490: Add security warning to TrustStrategy implementations documentation

2023-10-01 Thread via GitHub
Marcono1234 opened a new pull request, #490: URL: https://github.com/apache/httpcomponents-client/pull/490 Tries to improve the documentation of `TrustAllStrategy` and `TrustSelfSignedStrategy` by mentioning their security implications and suggesting more secure alternatives. For

[GitHub] [httpcomponents-core] Marcono1234 opened a new pull request, #438: Add security warning to TrustStrategy documentation

2023-10-01 Thread via GitHub
Marcono1234 opened a new pull request, #438: URL: https://github.com/apache/httpcomponents-core/pull/438 This tries to make the security aspects of `TrustStrategy` clearer and suggest that it might not be needed when self-signed certificates are used. Relates to #490 Any

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
arturobernalg commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342183980 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
arturobernalg commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342183980 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override

[GitHub] [httpcomponents-core] arturobernalg commented on a diff in pull request #437: HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5

2023-10-01 Thread via GitHub
arturobernalg commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342183980 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
ok2c commented on PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#issuecomment-1742799336 @Marcono1234 I also propose `TrustSelfSignedStrategy` be deprecated in favor of `SSLContextBuilder`. The class is basically useless. -- This is an automated message from the

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1342546547 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1342546547 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5 [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342526323 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override public

Re: [PR] HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5 [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c merged PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437 -- 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:

Re: [PR] HTTPCORE-756: replace CR, LF, NULL in header values with SP per RFC 9110 section 5.5 [httpcomponents-core]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #437: URL: https://github.com/apache/httpcomponents-core/pull/437#discussion_r1342526096 ## httpcore5/src/main/java/org/apache/hc/core5/http/message/BufferedHeader.java: ## @@ -110,7 +113,27 @@ public String getName() { @Override public

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-02 Thread via GitHub
ok2c commented on code in PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#discussion_r1342539092 ## httpclient5/src/main/java/org/apache/hc/client5/http/ssl/TrustAllStrategy.java: ## @@ -34,9 +34,15 @@ import org.apache.hc.core5.ssl.TrustStrategy;

[GitHub] [httpcomponents-core] karthik-119 opened a new pull request, #433: HTTPCORE-758-Fix exception handling when creating IO Dispatchers

2023-09-26 Thread via GitHub
karthik-119 opened a new pull request, #433: URL: https://github.com/apache/httpcomponents-core/pull/433 - Fix checkstyle errors to prevent build failures -- 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

[GitHub] [httpcomponents-client] dependabot[bot] commented on pull request #487: Bump httpcore.version from 5.2.2 to 5.2.3

2023-09-27 Thread via GitHub
dependabot[bot] commented on PR #487: URL: https://github.com/apache/httpcomponents-client/pull/487#issuecomment-1737445998 Looks like these dependencies are up-to-date now, so this is no longer needed. -- This is an automated message from the Apache Git Service. To respond to the

[GitHub] [httpcomponents-client] dependabot[bot] closed pull request #487: Bump httpcore.version from 5.2.2 to 5.2.3

2023-09-27 Thread via GitHub
dependabot[bot] closed pull request #487: Bump httpcore.version from 5.2.2 to 5.2.3 URL: https://github.com/apache/httpcomponents-client/pull/487 -- 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

[GitHub] [httpcomponents-client] ok2c commented on pull request #487: Bump httpcore.version from 5.2.2 to 5.2.3

2023-09-27 Thread via GitHub
ok2c commented on PR #487: URL: https://github.com/apache/httpcomponents-client/pull/487#issuecomment-1737443152 @dependabot rebase -- 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

[GitHub] [httpcomponents-client] arturobernalg opened a new pull request, #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status in HttpRequestRetryExec

2023-09-27 Thread via GitHub
arturobernalg opened a new pull request, #489: URL: https://github.com/apache/httpcomponents-client/pull/489 This PR updates the HttpRequestRetryExec class to disallow automatic retries when a 403 Forbidden status is received, in compliance with RFC 9110 guidelines. According to [RFC

[GitHub] [httpcomponents-client] arturobernalg commented on pull request #489: HTTPCLIENT-2293: Disallow Automatic Retries on 403 Forbidden Status

2023-09-27 Thread via GitHub
arturobernalg commented on PR #489: URL: https://github.com/apache/httpcomponents-client/pull/489#issuecomment-1738120965 Please @reschke take a look -- 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

[GitHub] [httpcomponents-core] ok2c closed pull request #433: HTTPCORE-758-Fix exception handling when creating IO Dispatchers

2023-09-26 Thread via GitHub
ok2c closed pull request #433: HTTPCORE-758-Fix exception handling when creating IO Dispatchers URL: https://github.com/apache/httpcomponents-core/pull/433 -- 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

[GitHub] [httpcomponents-core] ok2c commented on pull request #433: HTTPCORE-758-Fix exception handling when creating IO Dispatchers

2023-09-26 Thread via GitHub
ok2c commented on PR #433: URL: https://github.com/apache/httpcomponents-core/pull/433#issuecomment-1735615374 @karthik-119 Corrected by a5c117028b7c620974304636d52f06f172f1d08b -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-03 Thread via GitHub
ok2c merged PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438 -- 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:

Re: [PR] HTTPCLIENT-2293: Via header improvements [httpcomponents-client]

2023-10-03 Thread via GitHub
ok2c commented on PR #491: URL: https://github.com/apache/httpcomponents-client/pull/491#issuecomment-1745073739 @arturobernalg Please take a look -- 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

[PR] HTTPCLIENT-2293: Via header improvements [httpcomponents-client]

2023-10-03 Thread via GitHub
ok2c opened a new pull request, #491: URL: https://github.com/apache/httpcomponents-client/pull/491 * `ResponseCachingPolicy` optimized to generate less intermediate garbage while paring `Via` headers * `Via` request and response header generation moved from the caching exec

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-03 Thread via GitHub
ok2c merged PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490 -- 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:

Re: [PR] Add security warning to TrustStrategy implementations documentation [httpcomponents-client]

2023-10-03 Thread via GitHub
Marcono1234 commented on PR #490: URL: https://github.com/apache/httpcomponents-client/pull/490#issuecomment-1745324354 Have pushed the changes, I hope that is what you had in mind. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-03 Thread via GitHub
ok2c commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1344326091 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface can be

Re: [PR] Add security warning to TrustStrategy documentation [httpcomponents-core]

2023-10-03 Thread via GitHub
Marcono1234 commented on code in PR #438: URL: https://github.com/apache/httpcomponents-core/pull/438#discussion_r1344299429 ## httpcore5/src/main/java/org/apache/hc/core5/ssl/TrustStrategy.java: ## @@ -34,6 +34,19 @@ * configured in the actual SSL context. This interface

<    1   2   3   4   5   6   7   8   9   10   >