-
Notifications
You must be signed in to change notification settings - Fork 198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updates azd
to use default http client/transport
#3478
Conversation
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash:
pwsh:
WindowsPowerShell install
MSI install
Standalone Binary
MSIContainer
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I am inclined to accept. We should make sure and track down these http client issues. Filed: #3480
Updates
azd
to always return thehttp.DefaultClient
.Any transport/proxy settings required for the record/playback are also set on the default client
Note
What this means though is that somewhere in the app it is not using the correct http client instance. The recorder is catching more requests with this change than previously.