Skip to content

Commit d796783

Browse files
committed
feat(backend): update MCP installation logs permissions and descriptions
1 parent 524546c commit d796783

File tree

5 files changed

+8
-6
lines changed

5 files changed

+8
-6
lines changed

services/backend/api-spec.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13300,7 +13300,7 @@
1330013300
"tags": [
1330113301
"MCP Installations"
1330213302
],
13303-
"description": "Retrieves logs for a specific MCP installation with pagination and optional level filtering. Requires mcp.installations.view permission. Logs are returned newest first.",
13303+
"description": "Retrieves logs for a specific MCP installation with pagination and optional level filtering. Requires mcp.installations.logs.view permission. Logs are returned newest first.",
1330413304
"parameters": [
1330513305
{
1330613306
"schema": {

services/backend/api-spec.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9293,8 +9293,8 @@ paths:
92939293
tags:
92949294
- MCP Installations
92959295
description: Retrieves logs for a specific MCP installation with pagination and
9296-
optional level filtering. Requires mcp.installations.view permission.
9297-
Logs are returned newest first.
9296+
optional level filtering. Requires mcp.installations.logs.view
9297+
permission. Logs are returned newest first.
92989298
parameters:
92999299
- schema:
93009300
type: integer

services/backend/src/permissions/index.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ export const ROLE_DEFINITIONS = {
3535
'mcp.versions.manage',
3636
'mcp.installations.view_all',
3737
'mcp.installations.view',
38+
'mcp.installations.logs.view',
3839
'mcp.installations.create',
3940
'mcp.installations.edit',
4041
'mcp.installations.delete',
@@ -83,6 +84,7 @@ export const ROLE_DEFINITIONS = {
8384
'cloud_credentials.delete',
8485
'mcp.servers.read',
8586
'mcp.installations.view',
87+
'mcp.installations.logs.view',
8688
'mcp.installations.create',
8789
'mcp.installations.edit',
8890
'mcp.installations.delete',

services/backend/src/routes/teams/mcp-installations/logs-stream.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ export default async function getInstallationLogsStreamRoute(server: FastifyInst
7676
Querystring: LogsQueryParams;
7777
}>('/teams/:teamId/mcp/installations/:installationId/logs/stream', {
7878
sse: true,
79-
preValidation: requireTeamPermission('mcp.installations.view'),
79+
preValidation: requireTeamPermission('mcp.installations.logs.view'),
8080
schema: {
8181
tags: ['MCP Installations'],
8282
summary: 'Stream installation server logs (SSE)',

services/backend/src/routes/teams/mcp-installations/logs.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -159,12 +159,12 @@ export default async function getInstallationLogsRoute(server: FastifyInstance)
159159
Params: TeamAndInstallationParams;
160160
Querystring: LogsQueryParams;
161161
}>('/teams/:teamId/mcp/installations/:installationId/logs', {
162-
preValidation: requireTeamPermission('mcp.installations.view'),
162+
preValidation: requireTeamPermission('mcp.installations.logs.view'),
163163
schema: {
164164
tags: ['MCP Installations'],
165165
summary: 'Get installation logs',
166166
description:
167-
'Retrieves logs for a specific MCP installation with pagination and optional level filtering. Requires mcp.installations.view permission. Logs are returned newest first.',
167+
'Retrieves logs for a specific MCP installation with pagination and optional level filtering. Requires mcp.installations.logs.view permission. Logs are returned newest first.',
168168
security: [{ cookieAuth: [] }, { bearerAuth: [] }],
169169

170170
// Fastify validation schemas

0 commit comments

Comments
 (0)