Re: [PR] Use GET() and PUT() instead of method() [maven-resolver]

2025-06-11 Thread via GitHub
cstamas merged PR #740: URL: https://github.com/apache/maven-resolver/pull/740 -- 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: issues-unsubscr...@maven.

Re: [PR] Use GET() and PUT() instead of method() [maven-resolver]

2025-06-09 Thread via GitHub
cstamas commented on PR #740: URL: https://github.com/apache/maven-resolver/pull/740#issuecomment-2955302573 @slawekjaranowski like test "does JDK transport sends `Content-Length: 0` with GET? As otherwise JDK transport (as all HTTP transports, apache and jetty) are quite well covered. --

[PR] Use GET() and PUT() instead of method() [maven-resolver]

2025-06-08 Thread via GitHub
cstamas opened a new pull request, #740: URL: https://github.com/apache/maven-resolver/pull/740 But this does not work for HEAD() that is 18+. Moreover, builder `method(String, BodyPublisher)` enforces body publisher as non-null, while the logic to stop emitting `Content-Length: 0` on