turbot / steampipe-mod-microsoft365-compliance

Run individual controls or full compliance benchmarks for CIS across all of your Microsoft 365 and Office 365 tenants using Powerpipe and Steampipe.
https://hub.powerpipe.io/mods/turbot/microsoft365_compliance
Apache License 2.0
24 stars 5 forks source link

Controls should reference their query using `query =` rather than `sql =` #32

Closed madhushreeray30 closed 1 year ago

madhushreeray30 commented 1 year ago

It's simpler in code and readability to reference queries using the query = attribute rather than the sql = attribute.

This occurs throughout the mod, but as a specific example:

https://github.com/turbot/steampipe-mod-microsoft365-compliance/blob/a9cfb6fc654898c540b4296429a884b1a64e4fba/cis_v140/section_1.sp#L50-L63

Could be changed from:

sql           = query.azuread_admin_user_mfa_enabled.sql

to:

query         = query.azuread_admin_user_mfa_enabled
github-actions[bot] commented 1 year ago

'This issue is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 30 days.'

misraved commented 1 year ago

Closed as a part of https://github.com/turbot/steampipe-mod-microsoft365-compliance/pull/33 which is now available in v0.4 mod version