Skip to content

Commit ef3934c

Browse files
authored
fix nav link to be hidden and update access tag (#70607) (#70861)
1 parent 88428b7 commit ef3934c

File tree

5 files changed

+29
-16
lines changed

5 files changed

+29
-16
lines changed

x-pack/plugins/security_solution/common/constants.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,16 @@ export const DEFAULT_TIMEPICKER_QUICK_RANGES = 'timepicker:quickRanges';
3535
export const NO_ALERT_INDEX = 'no-alert-index-049FC71A-4C2C-446F-9901-37XMC5024C51';
3636
export const ENDPOINT_METADATA_INDEX = 'metrics-endpoint.metadata-*';
3737

38+
export enum SecurityPageName {
39+
alerts = 'alerts',
40+
overview = 'overview',
41+
hosts = 'hosts',
42+
network = 'network',
43+
timelines = 'timelines',
44+
case = 'case',
45+
management = 'management',
46+
}
47+
3848
export const APP_OVERVIEW_PATH = `${APP_PATH}/overview`;
3949
export const APP_ALERTS_PATH = `${APP_PATH}/alerts`;
4050
export const APP_HOSTS_PATH = `${APP_PATH}/hosts`;

x-pack/plugins/security_solution/public/app/types.ts

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -18,16 +18,8 @@ import { State, SubPluginsInitReducer } from '../common/store';
1818
import { Immutable } from '../../common/endpoint/types';
1919
import { AppAction } from '../common/store/actions';
2020
import { TimelineState } from '../timelines/store/timeline/types';
21+
export { SecurityPageName } from '../../common/constants';
2122

22-
export enum SecurityPageName {
23-
alerts = 'alerts',
24-
overview = 'overview',
25-
hosts = 'hosts',
26-
network = 'network',
27-
timelines = 'timelines',
28-
case = 'case',
29-
management = 'management',
30-
}
3123
export interface SecuritySubPluginStore<K extends SecuritySubPluginKeyStore, T> {
3224
initialState: Record<K, T | undefined>;
3325
reducer: Record<K, Reducer<T, AnyAction>>;

x-pack/plugins/security_solution/server/lib/timeline/routes/create_timelines_route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ export const createTimelinesRoute = (
3333
body: buildRouteValidation(createTimelineSchema),
3434
},
3535
options: {
36-
tags: ['access:siem'],
36+
tags: ['access:securitySolution'],
3737
},
3838
},
3939
async (context, request, response) => {

x-pack/plugins/security_solution/server/lib/timeline/routes/update_timelines_route.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,7 @@ export const updateTimelinesRoute = (
3131
body: buildRouteValidation(updateTimelineSchema),
3232
},
3333
options: {
34-
tags: ['access:siem'],
34+
tags: ['access:securitySolution'],
3535
},
3636
},
3737
// eslint-disable-next-line complexity

x-pack/plugins/security_solution/server/plugin.ts

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ import { initSavedObjects, savedObjectTypes } from './saved_objects';
3939
import { AppClientFactory } from './client';
4040
import { createConfig$, ConfigType } from './config';
4141
import { initUiSettings } from './ui_settings';
42-
import { APP_ID, APP_ICON, SERVER_APP_ID } from '../common/constants';
42+
import { APP_ID, APP_ICON, SERVER_APP_ID, SecurityPageName } from '../common/constants';
4343
import { registerEndpointRoutes } from './endpoint/routes/metadata';
4444
import { registerResolverRoutes } from './endpoint/routes/resolver';
4545
import { registerPolicyRoutes } from './endpoint/routes/policy';
@@ -70,6 +70,17 @@ export interface PluginSetup {}
7070
// eslint-disable-next-line @typescript-eslint/no-empty-interface
7171
export interface PluginStart {}
7272

73+
const securitySubPlugins = [
74+
APP_ID,
75+
`${APP_ID}:${SecurityPageName.overview}`,
76+
`${APP_ID}:${SecurityPageName.alerts}`,
77+
`${APP_ID}:${SecurityPageName.hosts}`,
78+
`${APP_ID}:${SecurityPageName.network}`,
79+
`${APP_ID}:${SecurityPageName.timelines}`,
80+
`${APP_ID}:${SecurityPageName.case}`,
81+
`${APP_ID}:${SecurityPageName.management}`,
82+
];
83+
7384
export class Plugin implements IPlugin<PluginSetup, PluginStart, SetupPlugins, StartPlugins> {
7485
private readonly logger: Logger;
7586
private readonly config$: Observable<ConfigType>;
@@ -144,12 +155,12 @@ export class Plugin implements IPlugin<PluginSetup, PluginStart, SetupPlugins, S
144155
}),
145156
order: 1100,
146157
icon: APP_ICON,
147-
navLinkId: 'securitySolution',
148-
app: ['securitySolution', 'kibana'],
158+
navLinkId: APP_ID,
159+
app: [...securitySubPlugins, 'kibana'],
149160
catalogue: ['securitySolution'],
150161
privileges: {
151162
all: {
152-
app: ['securitySolution', 'kibana'],
163+
app: [...securitySubPlugins, 'kibana'],
153164
catalogue: ['securitySolution'],
154165
api: ['securitySolution', 'actions-read', 'actions-all', 'alerting-read', 'alerting-all'],
155166
savedObject: {
@@ -177,7 +188,7 @@ export class Plugin implements IPlugin<PluginSetup, PluginStart, SetupPlugins, S
177188
],
178189
},
179190
read: {
180-
app: ['securitySolution', 'kibana'],
191+
app: [...securitySubPlugins, 'kibana'],
181192
catalogue: ['securitySolution'],
182193
api: ['securitySolution', 'actions-read', 'actions-all', 'alerting-read', 'alerting-all'],
183194
savedObject: {

0 commit comments

Comments
 (0)