-
Notifications
You must be signed in to change notification settings - Fork 309
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
azuread_conditional_access_policy: Implement support for the insider_risk_levels
parameter
#1597
Conversation
1a05bd9
to
784ee70
Compare
insider_risk_levels
parameter
internal/services/conditionalaccess/migrations/conditional_access_policy_resource.go
Outdated
Show resolved
Hide resolved
…ess_policy_resource.go Co-authored-by: stephybun <steph@hashicorp.com>
17e6aaa
to
f74fb14
Compare
@@ -251,13 +251,13 @@ func (NamedLocationResource) completeCountry(data acceptance.TestData) string { | |||
resource "azuread_named_location" "test" { | |||
display_name = "acctestNLC-%[1]d" | |||
country { | |||
country_lookup_method = "clientIpAddress" |
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.
Formatted acceptance test terraform blocks introduced in #1589 to pass the tflint
test.
@stephybun please take another look, I've fixed the document linter error. Edit: introduced in #1589 |
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.
Thanks @sdx-jkataja LGTM 💯
* Update CHANGELOG.md for #1585 include #1585 * Update CHANGELOG.md pulling in changes from duplicate changelog PR * Update CHANGELOG.md for #1619 * Update for #1589 * Update for #1593 * Update for #1592 * Update for #1597 * add changelog update script and updated changelog --------- Co-authored-by: stephybun <steph@hashicorp.com>
In
azuread_conditional_access_policy
add support for insider risk parameterinsider_risk_levels
. This allows specifying insider risk level with Conditional Access policies.Graph API Documentation: https://learn.microsoft.com/en-us/graph/api/resources/conditionalaccessconditionset?view=graph-rest-1.0
Fix #1521