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.

Sign up for GitHub

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

Jump to bottom

[Storage] Unsupported headers - proposal 2 - request conditions per api. #21205

Closed
wants to merge 1 commit into from
Closed

[Storage] Unsupported headers - proposal 2 - request conditions per api. #21205

wants to merge 1 commit into from

Conversation

kasobol-msft
Copy link
Contributor

@kasobol-msft kasobol-msft commented May 19, 2021 •
edited
Loading

In this PR: handle unsupported headers by having a type of request conditions per api.

Pros:

  • model maps 1:1 with service capabilities
  • evolution of APIs is decoupled

Cons:

  • explosion of model types

@ghost ghost added the Storage Storage Service (Queues, Blobs, Files) label May 19, 2021
@kasobol-msft kasobol-msft changed the title [Storage] Unsupported headers - proposal 1 - request conditions per api. [Storage] Unsupported headers - proposal 2 - request conditions per api. May 19, 2021
azure-sdk pushed a commit to azure-sdk/azure-sdk-for-net that referenced this pull request Nov 21, 2022
@jungph808 @dosiso
CodeGen from PR 21540 in Azure/azure-rest-api-specs
1fafeea 
Release securityinsights microsoft.security insights 2022 11 01 (Azure#21540)

* Adds base for updating Microsoft.SecurityInsights from version stable/2022-08-01 to version 2022-11-01

* Updates readme

* Updates API version in new specs and examples

* SecurityMLAnalyticsSettings to Microsoft.SecurityInsights-2022-11-01-stable (Azure#20944)

* update

* update api versions

* Automation rules 2022-11-01 - alert & update triggers (Azure#21205)

* Automation rules 2022-11-01 stable version

* preetier

* Remove unused url parameters

Co-authored-by: jungph808 <100548649+jungph808@users.noreply.github.com>
Co-authored-by: Dor Siso <60354892+dosiso@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seanmcc-msft seanmcc-msft Awaiting requested review from seanmcc-msft seanmcc-msft will be requested when the pull request is marked ready for review seanmcc-msft is a code owner

@amnguye amnguye Awaiting requested review from amnguye amnguye will be requested when the pull request is marked ready for review amnguye is a code owner

@jaschrep-msft jaschrep-msft Awaiting requested review from jaschrep-msft jaschrep-msft will be requested when the pull request is marked ready for review jaschrep-msft is a code owner

@schaabs schaabs Awaiting requested review from schaabs schaabs will be requested when the pull request is marked ready for review schaabs is a code owner

@tg-msft tg-msft Awaiting requested review from tg-msft tg-msft will be requested when the pull request is marked ready for review tg-msft is a code owner

@jalauzon-msft jalauzon-msft Awaiting requested review from jalauzon-msft jalauzon-msft will be requested when the pull request is marked ready for review jalauzon-msft is a code owner

@nickliu-msft nickliu-msft Awaiting requested review from nickliu-msft nickliu-msft will be requested when the pull request is marked ready for review nickliu-msft is a code owner

Assignees
No one assigned
Labels
Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kasobol-msft