Port Azure SDK patches to Consumption API v1.1.0 #1
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.
This PR ports two patches to the Azure SDK to address the following problems:
Add StartDate/EndDate parameters for MCA users
According to the Microsoft docs, modern customers with a Microsoft Customer Agreement (MCA) must use the
StartDate
andEndDate
query parameters to get the usage details for a specific date range.Unfortunately, the API spec does include these query parameters, so we are manually adding them to the generated client.
If we don't add the query parameters, MCA users will get (many more) usage details items outside the intended time range.
URL encode 'next link' in multipage responses
When there are more than 1000 usage details items, the Consumption API paginates the response. Each response page contains a 'next link' to the following page.
The
$filter
query parameters contain a date range expression like the following:Unfortunately, the Azure service adds the query parameters value to the 'next link' without URL-encoding them. Since one of the parameters,
$filter
contains space, it breaks the URL, causing the following query parameters to get lost causing the error.