azure-sdk-for-python icon indicating copy to clipboard operation
azure-sdk-for-python copied to clipboard

Check status in initial response for O-L and A-AO LRO flow

Open lmazuel opened this issue 3 years ago • 1 comments

For LRO, in presence of Operation-Location or Azure-AsyncOperation and a 2xx status code of the initial response, polling algorithm must check the “status” key for success. If success, do not poll on the header but return immediately (could be doing a GET on resourceLocation or returning the payload directly). This is a change of behavior from today, as right now the mere presence of Operation-Location or Azure-AsyncOperation by itself triggers a polling, irrespectively of which exact 2xx or “status” is received in the initial response.

lmazuel avatar Sep 21 '22 22:09 lmazuel

API change check

API changes are not detected in this pull request.

azure-sdk avatar Sep 21 '22 23:09 azure-sdk

Hi @lmazuel. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

ghost avatar Dec 02 '22 10:12 ghost

Hi @lmazuel. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing "/reopen" if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the "no-recent-activity" label; otherwise, this is likely to be closed again with the next cleanup pass.

ghost avatar Dec 09 '22 17:12 ghost