Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Api Export - Get : res.Value is nil #22556

Open
christapa opened this issue Mar 8, 2024 · 3 comments
Open

Api Export - Get : res.Value is nil #22556

christapa opened this issue Mar 8, 2024 · 3 comments
Assignees
Labels
API Management bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Service Attention This issue is responsible by Azure service team.

Comments

@christapa
Copy link

christapa commented Mar 8, 2024

Bug Report

  • import path of package in question
    github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/apimanagement/armapimanagement@v1.1.1

Same happen for :
github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/apimanagement/armapimanagement/v2

Expect (From the doc) :

{
  "id": "/subscriptions/subid/resourceGroups/rg1/providers/Microsoft.ApiManagement/service/apimService1/apis/echo-api",
  "format": "swagger-link-json",
  "value": {
    "link": "https://apimgmtstkjpszxxxxxxx.blob.core.windows.net/api-export/Swagger Petstore Extensive.json?sv=2015-07-08&sr=b&sig=mxhLsFuOonu8EXIjyFPV%2FnDra0qTIoip7N7MuU%2BTFsA%3D&se=2019-04-10T22:41:31Z&sp=r"
  }
}

Get (Code) :

{
  "id": "/subscriptions/xxxxx/resourceGroups/xxxxxx/providers/Microsoft.ApiManagement/service/xxxxx/apis/xxxx"
}
go 1.22.0

require (
	github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.5.1
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/apimanagement/armapimanagement v1.1.1
	github.com/golang-jwt/jwt/v5 v5.2.1
	github.com/microsoft/azure-devops-go-api/azuredevops/v7 v7.1.0
)

or V2

go 1.22.0

require (
	github.com/Azure/azure-sdk-for-go/sdk/azcore v1.10.0
	github.com/Azure/azure-sdk-for-go/sdk/azidentity v1.5.1
	github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/apimanagement/armapimanagement/v2 v2.1.0
	github.com/golang-jwt/jwt/v5 v5.2.1
	github.com/invopop/yaml v0.2.0
	github.com/sirupsen/logrus v1.9.3
)

@github-actions github-actions bot added API Management customer-reported Issues that are reported by GitHub users external to the Azure organization. Mgmt This issue is related to a management-plane library. needs-team-triage This issue needs the team to triage. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels Mar 8, 2024
@jhendrixMSFT jhendrixMSFT removed question The issue doesn't require a change to the product in order to be resolved. Most issues start as that needs-team-triage This issue needs the team to triage. labels Mar 8, 2024
@github-actions github-actions bot added the needs-team-attention This issue needs attention from Azure service team or SDK team label Mar 8, 2024
@lirenhe lirenhe assigned Alancere and unassigned lirenhe Mar 11, 2024
@Alancere
Copy link
Member

Alancere commented Mar 11, 2024

image
Server returns are inconsistent with the swagger definition

  • server return:
{
    "id": "apisId",
    "type": "Microsoft.ApiManagement/service/apis",
    "name": "apisName",
    "properties": {
        "format": "swagger-link-json",
        "value": {
            "link": "https://apims.blob.core.windows.net/api-export/Swagger Petstore.json?sv=2017-04-17&sr=b&sig="
        }
    }
}
  • swagger definiton:
{
    "id": "",
    "format": "",
    "value": {
        "link": ""
    }
}

@Alancere Alancere added the Service Attention This issue is responsible by Azure service team. label Mar 11, 2024
Copy link

github-actions bot commented Mar 11, 2024

Thanks for the feedback! We are routing this to the appropriate team for follow-up. cc @miaojiang, @solankisamir.

@Alancere
Copy link
Member

Hi @solankisamir, could you help.

@ronniegeraghty ronniegeraghty added the bug This issue requires a change to an existing behavior in the product in order to be resolved. label Mar 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
API Management bug This issue requires a change to an existing behavior in the product in order to be resolved. customer-reported Issues that are reported by GitHub users external to the Azure organization. Mgmt This issue is related to a management-plane library. needs-team-attention This issue needs attention from Azure service team or SDK team Service Attention This issue is responsible by Azure service team.
Projects
None yet
Development

No branches or pull requests

5 participants