Python msrestazure: LRO doesn't deserialize 'target' and 'details' fields on error

[yugangw-msft]

C# and node versions both handle it. The consequence of not doing it is the error surfaced is too vague
Azure/azure-cli#1007
For reference, the C# version is here:
https://github.com/Azure/autorest/blob/master/src/client/Microsoft.Rest.ClientRuntime.Azure/CloudError.cs#L34-L39

[annatisch]

Hi @yugangw-msft, are you able to confirm whether the response body returned by the webapp API conforms to the standard CloudError?
For example, I see that the C# implementation has Details as a list of CloudError objects, where as in the response Details has a list of key value pairs, only one of which is an "errorEntity".

[yugangw-msft]

webapp API does not conform to the standard CloudError. So you don't need to special case for webapp api, rather just get de-serialization logic in parity with C# and node

I am chatting with webapp team to see whether their service can fix the error body

[annatisch]

Thanks @yugangw-msft,
I think I've made some progress with this - one question - I notice that in the error response from webapp API in the CLI issue, all the fields are capitalized ("Code", "Message", etc).
Currently the CloudError deserialization in Python only looks for lower case fields. It appears from the node implementation that it may do so as well.
So - do you know if this should be completely case insensitive, or is this another formatting issue specific to the webapp API error response?

[yugangw-msft]

It should be lower case always, so webapp, again, goes against the guideline :(. @naveedaz

[lmazuel]

Fixed here: #1510

Release soon

[lmazuel]

Released in msrestazure 0.4.4