Skip to content

Commit fe889ec

Browse files
PhilippeObertiqn895
authored andcommitted
[Security Solution] remove riskScoringRoutesEnabled feature flag (elastic#240539)
## Summary This PR removes the `riskScoringRoutesEnabled` feature flag. This feature flag was enabled back in 2023 . **_Feel free to close this PR if our intention is to keep the feature flag around!_** > [!IMPORTANT] > As the feature flag has been eanbled for a while,, no UI or behavior changes should be introduced by this PR! ### Checklist - [x] The PR description includes the appropriate Release Notes section, and the correct `release_note:*` label is applied per the [guidelines](https://www.elastic.co/guide/en/kibana/master/contributing.html#kibana-release-notes-process) - [x] Review the [backport guidelines](https://docs.google.com/document/d/1VyN5k91e5OVumlc0Gb9RPa3h1ewuPE705nRtioPiTvY/edit?usp=sharing) and apply applicable `backport:*` labels.
1 parent cfdfe22 commit fe889ec

File tree

5 files changed

+15
-24
lines changed

5 files changed

+15
-24
lines changed

x-pack/solutions/security/plugins/security_solution/common/experimental_features.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,6 @@ export const allowedExperimentalValues = Object.freeze({
6060
*/
6161
newUserDetailsFlyoutManagedUser: false,
6262

63-
/**
64-
* Enables experimental Entity Analytics HTTP endpoints
65-
*/
66-
riskScoringRoutesEnabled: true,
67-
6863
/**
6964
* Disables ESQL-based risk scoring
7065
*/

x-pack/solutions/security/plugins/security_solution/public/entity_analytics/index.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,9 +11,9 @@ import { routes } from './routes';
1111
export class EntityAnalytics {
1212
public setup() {}
1313

14-
public start(isEnabled = false): SecuritySubPlugin {
14+
public start(): SecuritySubPlugin {
1515
return {
16-
routes: isEnabled ? routes : [],
16+
routes,
1717
};
1818
}
1919
}

x-pack/solutions/security/plugins/security_solution/public/management/links.ts

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,7 @@ import {
1515
} from '../../common/endpoint/service/authz';
1616
import {
1717
BLOCKLIST_PATH,
18+
ENDPOINT_EXCEPTIONS_PATH,
1819
ENDPOINTS_PATH,
1920
ENTITY_ANALYTICS_ENTITY_STORE_MANAGEMENT_PATH,
2021
ENTITY_ANALYTICS_MANAGEMENT_PATH,
@@ -23,25 +24,24 @@ import {
2324
MANAGE_PATH,
2425
POLICIES_PATH,
2526
RESPONSE_ACTIONS_HISTORY_PATH,
26-
SecurityPageName,
2727
SECURITY_FEATURE_ID,
28+
SecurityPageName,
2829
TRUSTED_APPS_PATH,
29-
ENDPOINT_EXCEPTIONS_PATH,
3030
TRUSTED_DEVICES_PATH,
3131
} from '../../common/constants';
3232
import {
3333
BLOCKLIST,
34+
ENDPOINT_EXCEPTIONS,
3435
ENDPOINTS,
36+
ENTITY_ANALYTICS_RISK_SCORE,
37+
ENTITY_STORE,
3538
EVENT_FILTERS,
3639
HOST_ISOLATION_EXCEPTIONS,
3740
MANAGE,
3841
POLICIES,
3942
RESPONSE_ACTIONS_HISTORY,
4043
TRUSTED_APPLICATIONS,
4144
TRUSTED_DEVICES,
42-
ENTITY_ANALYTICS_RISK_SCORE,
43-
ENTITY_STORE,
44-
ENDPOINT_EXCEPTIONS,
4545
} from '../app/translations';
4646
import { licenseService } from '../common/hooks/use_license';
4747
import type { LinkItem } from '../common/links/types';
@@ -218,7 +218,6 @@ export const links: LinkItem = {
218218
skipUrlState: true,
219219
hideTimeline: true,
220220
capabilities: [`${SECURITY_FEATURE_ID}.entity-analytics`],
221-
experimentalKey: 'riskScoringRoutesEnabled',
222221
licenseType: 'platinum',
223222
},
224223
{

x-pack/solutions/security/plugins/security_solution/public/plugin.tsx

Lines changed: 8 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -7,15 +7,15 @@
77

88
import React from 'react';
99
import { i18n } from '@kbn/i18n';
10-
import { BehaviorSubject, Subject, combineLatestWith } from 'rxjs';
10+
import { BehaviorSubject, combineLatestWith, Subject } from 'rxjs';
1111
import type * as H from 'history';
1212
import type {
1313
AppMountParameters,
1414
AppUpdater,
1515
CoreSetup,
1616
CoreStart,
17-
PluginInitializerContext,
1817
Plugin as IPlugin,
18+
PluginInitializerContext,
1919
} from '@kbn/core/public';
2020
import { AppStatus, DEFAULT_APP_CATEGORIES } from '@kbn/core/public';
2121
import { Storage } from '@kbn/kibana-utils-plugin/public';
@@ -30,20 +30,20 @@ import type {
3030
PluginSetup,
3131
PluginStart,
3232
SetupPlugins,
33+
StartedSubPlugins,
3334
StartPlugins,
35+
StartPluginsDependencies,
3436
StartServices,
3537
SubPlugins,
36-
StartedSubPlugins,
37-
StartPluginsDependencies,
3838
} from './types';
39-
import { SOLUTION_NAME, ASSISTANT_MANAGEMENT_TITLE } from './common/translations';
39+
import { ASSISTANT_MANAGEMENT_TITLE, SOLUTION_NAME } from './common/translations';
4040

41-
import { APP_ID, APP_UI_ID, APP_PATH, APP_ICON_SOLUTION } from '../common/constants';
41+
import { APP_ICON_SOLUTION, APP_ID, APP_PATH, APP_UI_ID } from '../common/constants';
4242

4343
import type { AppLinkItems } from './common/links';
4444
import {
45-
applicationLinksUpdater,
4645
type ApplicationLinksUpdateParams,
46+
applicationLinksUpdater,
4747
} from './app/links/application_links_updater';
4848
import type { FleetUiExtensionGetterOptions, SecuritySolutionUiConfigType } from './common/types';
4949

@@ -344,9 +344,7 @@ export class Plugin implements IPlugin<PluginSetup, PluginStart, SetupPlugins, S
344344
rules: subPlugins.rules.start(storage),
345345
threatIntelligence: subPlugins.threatIntelligence.start(),
346346
timelines: subPlugins.timelines.start(),
347-
entityAnalytics: subPlugins.entityAnalytics.start(
348-
this.experimentalFeatures.riskScoringRoutesEnabled
349-
),
347+
entityAnalytics: subPlugins.entityAnalytics.start(),
350348
siemMigrations: subPlugins.siemMigrations.start(this.experimentalFeatures),
351349
siemReadiness: subPlugins.siemReadiness.start(),
352350
configurations: subPlugins.configurations.start(),

x-pack/solutions/security/test/security_solution_api_integration/config/ess/config.base.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -108,7 +108,6 @@ export function createTestConfig(options: CreateTestConfigOptions, testFiles?: s
108108
'--xpack.ruleRegistry.unsafe.legacyMultiTenancy.enabled=true',
109109
`--xpack.securitySolution.enableExperimental=${JSON.stringify([
110110
'previewTelemetryUrlEnabled',
111-
'riskScoringRoutesEnabled',
112111
'bulkEditAlertSuppressionEnabled',
113112
'doesNotMatchForIndicatorMatchRuleEnabled',
114113
'endpointExceptionsMovedUnderManagement',

0 commit comments

Comments
 (0)