search

Azure / azure-sdk-for-go

This repository is for active development of the Azure SDK for Go. For consumers of the SDK we recommend visiting our public developer docs at:
https://docs.microsoft.com/azure/developer/go/
MIT License
1.63k stars 832 forks source link

Bug Report: `Resource ID` is missing in API Management Logger response #4211

Closed JunyiYi closed 5 years ago

JunyiYi commented 5 years ago

Bug Report

Resource ID is missing in API Management Logger response.

tombuildsstuff commented 5 years ago

@JunyiYi it looks like it's available in the top level object: https://github.com/Azure/azure-sdk-for-go/blob/master/services/apimanagement/mgmt/2018-01-01/apimanagement/models.go#L5485 :)

jhendrixMSFT commented 5 years ago

Tom is correct, please see here https://github.com/Azure/azure-sdk-for-go/blob/master/services/apimanagement/mgmt/2018-01-01/apimanagement/models.go#L5485

JunyiYi commented 5 years ago

Hi @tombuildsstuff @jhendrixMSFT , it is my fault of not explaining it clearly. The Resource ID here represents properties.resourceId, not the ID of the logger.

For example, in the response body of this example, I mean the resourceId property is missing.

{
  "id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.ApiManagement/service/apimService1/loggers/loggerId",
  ...
  "properties": {
    ...
    "resourceId": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.EventHub/namespaces/hydraeventhub-ns"
  }
}
jhendrixMSFT commented 5 years ago

Thanks for clarifying. Since this is an issue with the swagger I've ported this issue to https://github.com/Azure/azure-rest-api-specs/issues/5318