-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
EventGrid: New preview API version with support for IPFiltering in Domains & Topics. #8100
Conversation
…rt in Domains & Topics. EventGrid: New preview API version with support for IPFiltering support in Domains & Topics. oTopicProperties: Following new properties are introduced: allowTrafficFromAllIPs inboundIpRules o DomainProperties: Following new properties are introduced: allowTrafficFromAllIPs inboundIpRules o Minor change in the description fields: o InboundIpRule: Following new model with the following properties are introduced: IpMask Action
Update Examples for Domain/Topics
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-java - Release
|
azure-sdk-for-js - Release
|
Azure Pipelines successfully started running 1 pipeline(s). |
azure-sdk-for-net - Release
|
azure-sdk-for-python - Release
|
azure-sdk-for-go - Release
|
Azure Pipelines successfully started running 1 pipeline(s). |
1 similar comment
Azure Pipelines successfully started running 1 pipeline(s). |
Correct the misspelled description
Azure Pipelines successfully started running 1 pipeline(s). |
Fix Prettier issue
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
2 similar comments
Azure Pipelines successfully started running 1 pipeline(s). |
Azure Pipelines successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few questions, everything else looks good.
"200": { | ||
"description": "OK", | ||
"schema": { | ||
"$ref": "#/definitions/TopicsListResult |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks like you removed 'readonly': 'true'.
Was this intentional?
...ion/eventgrid/resource-manager/Microsoft.EventGrid/preview/2020-04-01-preview/EventGrid.json
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
compared with previous 2020-01-01-preview.
minimal changes.
lgtm
EventGrid: New preview API version with support for IPFiltering in Domains & Topics.
oTopicProperties: Following new properties are introduced:
allowTrafficFromAllIPs
inboundIpRules
oDomainProperties: Following new properties are introduced:
allowTrafficFromAllIPs
inboundIpRules
oMinor change in the description fields:
oInboundIpRule: Following new model with the following properties are introduced:
IpMask
Action
Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub - one location for using our validation tools and finding your workflow.
Contribution checklist:
ARM API Review Checklist
Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.
Please follow the link to find more details on API review process.