Azure / Azure/azure-rest-api-specs

Invalid id in response of Monitor-DiagnosticSetting-SubscriptionDiagnosticSetting: it doesn't start with "/"

Open
#17,839 1 comment 0 reactions 0 assignees View on GitHub
Mgmt Monitor - Diagnostic Settings Service Attention
Dominant language
TypeSpec
Stars
3.1k
Forks
5.9k
Avg merge
3d 2h
Merged PRs (30d)
424

Description

The response of GET or PUT or other types of operation, SubscriptionDiagnosticSetting's `id` doesn't start with `/`
e.g. :
https://github.com/Azure/azure-rest-api-specs/blob/24b1f9ed9a0768a55b14cced1761a40f7c12b6d2/specification/monitor/resource-manager/Microsoft.Insights/preview/2017-05-01-preview/examples/getSubscriptionDiagnosticSetting.json#L11

Normally, ARM Resource's `id` should start with `/`. This is a blocking issue for Monitor Track2 .NET SDK.

Contributor guide

Open the contributing guide

Research direction

Start with the linked file, specification/monitor/resource-manager/Microsoft.Insights/preview/2017-05-01-preview/examples/getSubscriptionDiagnosticSetting.json, at line 11. Compare the SubscriptionDiagnosticSetting response with the ARM resource ID convention, then inspect the related GET, PUT, and other examples; done means the affected response IDs consistently begin with "/".

Written by the indexing model from the issue text.

Assessment

Tech stack
openapi
Domain
api, cloud
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.