Adding explicit HTTP error guidance #1373
Draft
+12
−8
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adding explicit guidance for returning HTTP errors when input is not acceptable, explicitly requiring a 406 status code for unmet
Accept
header requirements and 400 status for unmet request body requirements.Motivation and Context
There's varying behavior among SDKs when handling specified
Accept
header and request body requirements, with at least one returning HTTP status 500 (Internal Server Error) errors instead of 406 or 400 respectively. Status 500 should be used when the reason isn't known, as opposed to cases where the reason is known through explicit header and request body validation.How Has This Been Tested?
Generated, checked, and verified local preview.
Breaking Changes
Some SDKs will need to update the error they return in the explicitly defined cases.
Types of changes
Checklist