Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-20 Thread via GitHub
arturobernalg commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1864849429 > @arturobernalg I am going to revert this change-set for now and proceed with `5.3-alpha1` release. We can revisit this issue after the release. @ok2c fine by me.

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-20 Thread via GitHub
ok2c commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1864825368 @arturobernalg I am going to revert this change-set for now and proceed with `5.3-alpha1` release. We can revisit this issue after the release. -- This is an automated message f

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
michael-o commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1863303032 > https://www.rfc-editor.org/rfc/rfc9110.html#section-4.2.4 is not about HTTP(s) URIs in general, but what should happen WRT userinfo in an HTTP message (be it in the target

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
reschke commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1863101407 https://www.rfc-editor.org/rfc/rfc9110.html#section-4.2.4 is not about HTTP(s) URIs in general, but what should happen WRT userinfo in an HTTP message (be it in the target URI)

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
ok2c commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1863020528 @arturobernalg @michael-o I propose this change-set be reverted so I could proceed with the `5.3-alpha1` release. We can revisit the issue in the course of `5.3-alpha2`. -- Thi

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
michael-o commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1862828439 Let me re-read 4.2.4 -- 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 speci

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
michael-o commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1862818751 Good question. @reschke to the rescue -- 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 t

Re: [PR] HTTPCORE-756 - Deprecation of userInfo in Compliance with RFC 9110 [httpcomponents-core]

2023-12-19 Thread via GitHub
ok2c commented on PR #418: URL: https://github.com/apache/httpcomponents-core/pull/418#issuecomment-1862776884 @arturobernalg @michael-o As I suspected the complete removal of `userinfo` support turned out to be not such a good idea. Now HttpClient is unable to reject requests with URIs co