Skip to content

Conversation

@aruneko
Copy link
Contributor

@aruneko aruneko commented Feb 7, 2023

Summary

I found bugs in Azure CIS Benchmark 1.3.0 queries about MS Defender rules in Section 2.

Current queries check the pricing_tier (snake case) field in the properties column in the azure_security_pricings table. But the actual data format is below.

{"subPlan": "PerTransaction", "pricingTier": "Standard", "freeTrialRemainingTime": "PT0S"}

That's why, those queries always return fail since the desired field is pricingTier (camel case).

@cq-bot cq-bot added the azure label Feb 7, 2023
@aruneko aruneko changed the title Fix Azure CIS Policies in Section 2 fix(azure) Fix Azure CIS Policies in Section 2 Feb 7, 2023
@aruneko aruneko changed the title fix(azure) Fix Azure CIS Policies in Section 2 fix(azure): Fix Azure CIS Policies in Section 2 Feb 7, 2023
Copy link
Member

@erezrokah erezrokah left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great find @aruneko, thanks for the PR 🎸

@erezrokah erezrokah added the automerge Automatically merge once required checks pass label Feb 7, 2023
@kodiakhq kodiakhq bot merged commit 3c77b2c into cloudquery:main Feb 7, 2023
@aruneko aruneko deleted the fix/azure_cis_sec2_policies branch February 7, 2023 09:17
kodiakhq bot pushed a commit that referenced this pull request Feb 7, 2023
🤖 I have created a release *beep* *boop*
---


## [4.1.0](plugins-source-azure-v4.0.0...plugins-source-azure-v4.1.0) (2023-02-07)


### Features

* **azure-resources:** Add Virtual Network Gateways and Connections ([#7636](#7636)) ([97a9254](97a9254))


### Bug Fixes

* **azure-resources:** Use default API version of `armhealthbot` bots client ([#7177](#7177)) ([e24af94](e24af94))
* **azure:** Fix Azure CIS Policies in Section 2 ([#7718](#7718)) ([3c77b2c](3c77b2c))
* **deps:** Update golang.org/x/exp digest to f062dba ([#7531](#7531)) ([59d5575](59d5575))
* **deps:** Update google.golang.org/genproto digest to 1c01626 ([#7533](#7533)) ([c549c27](c549c27))
* **deps:** Update module github.com/Azure/azure-sdk-for-go/sdk/azidentity to v1.2.1 ([#7540](#7540)) ([3b5c838](3b5c838))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.33.0 ([#7595](#7595)) ([c5adc75](c5adc75))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.33.1 ([#7614](#7614)) ([2fe665c](2fe665c))
* **deps:** Update module github.com/cloudquery/plugin-sdk to v1.34.0 ([#7719](#7719)) ([6a33085](6a33085))
* **deps:** Update module github.com/golang-jwt/jwt/v4 to v4.4.3 ([#7543](#7543)) ([0607454](0607454))

---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Automatically merge once required checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants