Skip to content

Commit

Permalink
Remove securityRulesCancelEnabled setting and set shorter default tim…
Browse files Browse the repository at this point in the history
…eouts (#129769) (#130048)

(cherry picked from commit 5fb9576)

Co-authored-by: Marshall Main <55718608+marshallmain@users.noreply.github.com>
  • Loading branch information
kibanamachine and marshallmain authored Apr 12, 2022
1 parent 64b05e6 commit 4428b67
Show file tree
Hide file tree
Showing 7 changed files with 2 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@ export const allowedExperimentalValues = Object.freeze({
disableIsolationUIPendingStatuses: false,
riskyHostsEnabled: false,
riskyUsersEnabled: false,
securityRulesCancelEnabled: false,
pendingActionResponsesWithAck: true,
policyListEnabled: true,

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ export const createEqlAlertType = (
return {
id: EQL_RULE_TYPE_ID,
name: 'Event Correlation Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
validate: {
params: {
validate: (object: unknown) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ export const createIndicatorMatchAlertType = (
return {
id: INDICATOR_RULE_TYPE_ID,
name: 'Indicator Match Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
ruleTaskTimeout: '1h',
validate: {
params: {
validate: (object: unknown) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -16,11 +16,10 @@ import { CreateRuleOptions, SecurityAlertType } from '../types';
export const createMlAlertType = (
createOptions: CreateRuleOptions
): SecurityAlertType<MachineLearningRuleParams, {}, {}, 'default'> => {
const { logger, ml, experimentalFeatures } = createOptions;
const { logger, ml } = createOptions;
return {
id: ML_RULE_TYPE_ID,
name: 'Machine Learning Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
validate: {
params: {
validate: (object: unknown) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@ export const createQueryAlertType = (
return {
id: QUERY_RULE_TYPE_ID,
name: 'Custom Query Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
validate: {
params: {
validate: (object: unknown) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,6 @@ export const createSavedQueryAlertType = (
return {
id: SAVED_QUERY_RULE_TYPE_ID,
name: 'Saved Query Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
validate: {
params: {
validate: (object: unknown) => {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ export const createThresholdAlertType = (
return {
id: THRESHOLD_RULE_TYPE_ID,
name: 'Threshold Rule',
ruleTaskTimeout: experimentalFeatures.securityRulesCancelEnabled ? '5m' : '1d',
validate: {
params: {
validate: (object: unknown): ThresholdRuleParams => {
Expand Down

0 comments on commit 4428b67

Please sign in to comment.