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.

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

Fix matrix dimension length bug for string values #18231

Merged
merged 1 commit into from
Jan 27, 2021

Conversation

benbp
Copy link
Member

@benbp benbp commented Jan 27, 2021

This fixes a bug introduced in the last update where we get the incorrect matrix dimension length when the value is a singular string (as opposed to an array or object grouping).

This also fixes an issue where a display name filter might make a matrix empty when passed to the regex filter step.

@benbp benbp self-assigned this Jan 27, 2021
@benbp benbp added the Central-EngSys This issue is owned by the Engineering System team. label Jan 27, 2021
@benbp benbp merged commit f7d760c into Azure:master Jan 27, 2021
@benbp benbp deleted the benbp/matrix-dimension-bug branch January 27, 2021 20:05
openapi-sdkautomation bot pushed a commit to AzureSDKAutomation/azure-sdk-for-net that referenced this pull request Mar 30, 2022
dev-Sentinel-2022-04-01-preview (Azure#18410)

* Adds base for updating Microsoft.SecurityInsights from version preview/2022-01-01-preview to version 2022-04-01-preview

* Updates readme

* Updates API version in new specs and examples

* Fix ThreatInteliignece accordig to latest preview for alignment (Azure#18231)

* Add UebaEntityProviders to EntityAnalyticsSettings (Azure#18196)

* Add UebaEntityProviders to EntityAnalyticsSettings

* remove ueba

* Added additionalProperties to itemsKeyValue to resolve autorest issues (Azure#18170)

* added additionalProperties to itemsKeyValue to resolve autorest issues

* Update Watchlists.json

* ran prettier-fix

* Remove readonly property from ownerType (Azure#18450)

Co-authored-by: dosegal <[email protected]>
Co-authored-by: NoaGoren <[email protected]>
Co-authored-by: ushasan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Central-EngSys This issue is owned by the Engineering System team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants