Skip to content

Revert "Move the connection back to the connection pool when HTTP error 404 was received (#5308)"#5472

Merged
antkmsft merged 3 commits intoAzure:mainfrom
antkmsft:revert-curl-connpool-404
Apr 3, 2024
Merged

Revert "Move the connection back to the connection pool when HTTP error 404 was received (#5308)"#5472
antkmsft merged 3 commits intoAzure:mainfrom
antkmsft:revert-curl-connpool-404

Conversation

@antkmsft
Copy link
Copy Markdown
Member

@antkmsft antkmsft commented Mar 30, 2024

This reverts commit 79cc06d / PR #5308.

Fixes #5450, although PR #5473 should fix it better.
I propose to release this change as GA patch/minor, and then immediately release #5473 as Beta first.

@antkmsft antkmsft self-assigned this Mar 30, 2024
@antkmsft antkmsft requested a review from LarryOsterman March 30, 2024 06:24
@antkmsft antkmsft marked this pull request as ready for review March 30, 2024 09:07
@Azure Azure deleted a comment from azure-pipelines Bot Mar 30, 2024
@Azure Azure deleted a comment from azure-pipelines Bot Mar 30, 2024
@antkmsft antkmsft requested a review from Jinming-Hu April 2, 2024 04:12
@antkmsft
Copy link
Copy Markdown
Member Author

antkmsft commented Apr 2, 2024

FYI @Jinming-Hu, in case you missed it.
As I described in both PRs, I want to release this as non-Beta on Thursday, following #5473 being released as Beta on Thursday as well.

@antkmsft
Copy link
Copy Markdown
Member Author

antkmsft commented Apr 2, 2024

FYI @mchelnokov

@antkmsft antkmsft merged commit d62b0d6 into Azure:main Apr 3, 2024
@antkmsft antkmsft deleted the revert-curl-connpool-404 branch April 3, 2024 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HTTP Error 400. The request verb is invalid

2 participants