Azure / Azure/azure-sdk-tools

[Breaking Change][False negative] Response header change from `Azure-AsyncOperation` to `Location` not detected.

Open
#6,241 0 comments 0 reactions 1 assignee Claimed by @konrad-jamrozik View on GitHub
Breaking Changes bug Central-EngSys Spec PR Tools
Dominant language
C#
Stars
135
Forks
260
Avg merge
3d 1h
Merged PRs (30d)
143

Description

Full context here in this comment:
- https://github.com/Azure/azure-rest-api-specs-pr/pull/12202#discussion_r1203579311

### Issue summary

Compare [before](https://github.com/Azure/azure-rest-api-specs-pr/blob/794ee543d1bcf874105dde2528157e71ffe85fac/specification/azure-kusto/resource-manager/Microsoft.Kusto/stable/2022-12-29/kusto.json#L184) with [after](https://github.com/Azure/azure-rest-api-specs-pr/blob/794ee543d1bcf874105dde2528157e71ffe85fac/specification/azure-kusto/resource-manager/Microsoft.Kusto/stable/2023-05-02/kusto.json#L190):

before:
```json
"201": {
"description": "Successfully updated the cluster.",
"headers": {
"Azure-AsyncOperation": {
```

after:
```json
"201": {
"description": "Successfully updated the cluster.",
"headers": {
"Location": {
```

This should have been detected by our breaking change detector, but wasn't.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.