yknoya opened a new pull request, #12766: URL: https://github.com/apache/trafficserver/pull/12766
# Background This PR is derived from https://github.com/apache/trafficserver/pull/12729. Since the changes in that PR mixed a bug fix and cleanup work, we decided to split them into two separate PRs. This PR extracts and focuses on the cleanup portion. The motivation for this cleanup is that pre-setting `EIO` in `connect_result` before a connection attempt is error-prone. Instead, we want to set an error in `connect_result` only when a connection error actually occurs. This work was triggered by the following review comment: https://github.com/apache/trafficserver/pull/12729#discussion_r2601216919 > [!NOTE] > For the background on why `EIO` started being pre-set in `connect_result` before connection attempts, see https://github.com/apache/trafficserver/pull/7580 # Cleanup details See the following comment for details: https://github.com/apache/trafficserver/pull/12729#discussion_r2618628728 -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
