[PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread via GitHub
ok2c opened a new pull request, #494: URL: https://github.com/apache/httpcomponents-core/pull/494 Folks I just recently discovered [TestContainers](https://java.testcontainers.org/) and it this framework is an absolute marvel! I took it as an opportunity to revise and improve

Re: [PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread via GitHub
garydgregory commented on PR #494: URL: https://github.com/apache/httpcomponents-core/pull/494#issuecomment-2404826384 I use TextContainers elsewhere, works like champ! -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread via GitHub
ok2c commented on code in PR #494: URL: https://github.com/apache/httpcomponents-core/pull/494#discussion_r1795854034 ## httpcore5-testing/src/test/java/org/apache/hc/core5/testing/compatibility/ContainerImages.java: ## @@ -0,0 +1,94 @@ +/* + * =

Re: [PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread via GitHub
arturobernalg commented on code in PR #494: URL: https://github.com/apache/httpcomponents-core/pull/494#discussion_r1795864799 ## httpcore5-testing/src/test/java/org/apache/hc/core5/testing/compatibility/ContainerImages.java: ## @@ -0,0 +1,94 @@ +/* + *

Re: [PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread via GitHub
massdosage commented on code in PR #494: URL: https://github.com/apache/httpcomponents-core/pull/494#discussion_r1795738709 ## httpcore5-testing/src/test/java/org/apache/hc/core5/testing/compatibility/ContainerImages.java: ## @@ -0,0 +1,94 @@ +/* + * ===

Re: [PR] Migrated Docker based integration tests to TestContainers [httpcomponents-core]

2024-10-10 Thread Gary Gregory
I agree that "latest" doesn't sound reproducible but more importantly is not secure, this has been an issue in other ecosystems. We pin versions of dependencies in POMs, we should do it here as well. Good find! Gafy On Thu, Oct 10, 2024, 12:12 PM massdosage (via GitHub) wrote: > > massdosage co

Re: [PR] Properly alert receivers when using non-default dynamic table sizes for HPACK [httpcomponents-core]

2024-10-10 Thread via GitHub
CelestiaTheDryad commented on PR #495: URL: https://github.com/apache/httpcomponents-core/pull/495#issuecomment-2405952333 Rebased onto master. -- 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

Re: [PR] Properly alert receivers when using non-default dynamic table sizes for HPACK [httpcomponents-core]

2024-10-10 Thread via GitHub
ok2c commented on PR #495: URL: https://github.com/apache/httpcomponents-core/pull/495#issuecomment-2406045571 @CelestiaTheDryad This is big. I will need some time to review the changes. Do you think there is a possibility to put together a test case to verify the fix? -- This is an au