Description
openedon Apr 24, 2023
- @azure/digital-twins-core:
- ^1.1.0:
- Operating system:
- nodejs
- 16.13:
- browser
- name/version:
- typescript
- ^4.3.5:
- Is the bug related to documentation in
- README.md
- source code documentation
- SDK API docs on https://docs.microsoft.com
Describe the bug
Hey. While working with digital twins client and trying to execute few queries at same time I expected that options passed to DigitalTwinsClient
:
retryOptions: {
maxRetries: 20,
retryDelayInMs: 5000,
},
don't change anything. My queries got immediate rejected with QuotaExceptionError
. After some investigation of your code I spot that also there is no Retry-After
header in response as mentioned in the docs. I found it in @azure/core-rest-pipeline/dist/index.js@getRetryAfterInMs
.
if (!(response && [429, 503].includes(response.status)))
return undefined;
Status code is 200 for QuotaExceptionError
.
const retryAfterHeader = response.headers.get(RetryAfterHeader);
There is no "retry-after-ms", "x-ms-retry-after-ms", "Retry-After"
in response.
To Reproduce
Steps to reproduce the behavior:
- Execute few DT queries using
digitalTwinsClient.queryTwins()
till you will getQuotaExceptionError
.
Expected behavior
Retry logic is working.
Screenshots
Here is function which I modified.
Additional context
Add any other context about the problem here.