oav icon indicating copy to clipboard operation
oav copied to clipboard

Improvement on LRO_RESPONSE_HEADER rule for data-planes swagger

Open raych1 opened this issue 2 years ago • 2 comments

Data-plane swaggers has different requirement for long running operation than ARM swaggers.

As talked with @weidongxu-microsoft offline, the rule will be allowing Operation-Id or Operation-Location headers at least one of them should be present as long as the operation is declared as LRO.

raych1 avatar Nov 04 '22 02:11 raych1

The operation-location header must be present. operation-id must be present if present in the input and should be present otherwise. But I assume the existing rule is marked as arm-only (rpc), right? If so, why is it triggered for data planes?

johanste avatar Nov 07 '22 20:11 johanste

The rule might be intended to work for both data-plane and mgmt-plane, but its current implementation only handle for mgmt.

So I assume the fix is either to make it work for data-plane (I think Ray is targeting this), or just don't run it for data-plane.

weidongxu-microsoft avatar Nov 08 '22 02:11 weidongxu-microsoft