-
Notifications
You must be signed in to change notification settings - Fork 389
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
[datadog_security_monitoring_rule] Update provider for detection method impossible travel #1402
Conversation
c721f8a
to
1c5f46d
Compare
Updates the provider for security monitoring rules to accept rules with the new detection method `impossible_travel`. See also: - DataDog/documentation#13204 - DataDog/datadog-api-client-go#1357
1c5f46d
to
37827a9
Compare
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 good, one copy nit!
|
||
Optional: | ||
|
||
- **baseline_user_locations** (Boolean) If true, signals are suppressed for the first 24 hours. In that time, Datadog learns the user's regular access locations. This can be helpful to reduce noise and infer VPN usage or credentialed API access. Default is `false`. |
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.
- **baseline_user_locations** (Boolean) If true, signals are suppressed for the first 24 hours. In that time, Datadog learns the user's regular access locations. This can be helpful to reduce noise and infer VPN usage or credentialed API access. Default is `false`. | |
- **baseline_user_locations** (Boolean) If true, signals are suppressed for the first 24 hours. Datadog uses this time to learn about the user's regular access locations. This is helpful to reduce noise and infer VPN usage or credentialed API access. Default is `false`. |
metric = "@usr.handle" | ||
group_by_fields = ["@usr.handle"] | ||
} | ||
|
||
case { | ||
name = "" | ||
status = "high" | ||
notifications = ["@user"] | ||
} | ||
|
||
options { | ||
detection_method = "impossible_travel" | ||
keep_alive = 600 | ||
max_signal_duration = 900 | ||
impossible_travel_options { | ||
baseline_user_locations = true | ||
} | ||
} |
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.
@muffix can you please reformat it?
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.
Done, thanks
metric = "@usr.handle" | ||
group_by_fields = ["@usr.handle"] | ||
} | ||
|
||
case { | ||
name = "new case name (updated)" | ||
status = "high" | ||
notifications = ["@user"] | ||
} | ||
|
||
options { | ||
detection_method = "impossible_travel" | ||
keep_alive = 600 | ||
max_signal_duration = 900 | ||
impossible_travel_options { | ||
baseline_user_locations = true | ||
} |
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.
@muffix can you please reformat it?
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.
Done, thanks
Updates the provider for security monitoring rules to accept rules with
the new detection method
impossible_travel
.See also: