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
Revert "Adjust PUT method behavior to POST one for default content type in WebCmdlets" #21049
Revert "Adjust PUT method behavior to POST one for default content type in WebCmdlets" #21049
Conversation
…pe in WebCmdlets"
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? 👍 :ok_hand: :thumbsdown: (Email) |
There was another comment from @CarloToso #16318 (comment)
Original request from #16318 was that all method should work identically. Single difference from POST and PUT is #16318 (comment)
Adding the Content Type to the PUT was an attempt to avoid regression in PowerShell scenarios compared to removing it from the POST. If we still see regression, then we return to the original question of how to make methods with one behavior - remove ContentType from POST as in curl? And do we need to leave this default value there for hashtables as specific PowerShell scenario? This change was made almost a year(!) ago at the beginning of milestone. I'm surprised why the team spends so much effort on releasing monthly previews if no one uses them even with Azure. |
@iSazonov agree that this should have been found earlier and I've given the reporting team feedback to be actively testing against our previews. When did we add the content-type to POST? |
Feb 15, 2022 #19152 |
The @PowerShell/wg-powershell-cmdlets reviewed this. Given that there hasn't been reports of experienced regression of the change in POST behavior with regards to the |
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.
LGTM
This comment was marked as off-topic.
This comment was marked as off-topic.
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.
Thanks
Reverts #19152
This has been reported as a regression for the cmdlet working against Azure Blob Storage. The added
content-type
withPUT
is causing the request to be rejected. It seems thatcurl
does add thecontent-type
forPOST
, but not forPUT
, so we should do the same with the cmdlets.