-
Notifications
You must be signed in to change notification settings - Fork 5.6k
EventGrid: New preview API version with support for IPFiltering in Domains & Topics. #8100
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
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). |
ryansbenson
left a comment
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
zikalino
left a comment
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.