Skip to content

Apply security policy to flow endpoints#2098

Merged
MarceloRGonc merged 3 commits intomainfrom
mg/flows
Mar 10, 2026
Merged

Apply security policy to flow endpoints#2098
MarceloRGonc merged 3 commits intomainfrom
mg/flows

Conversation

@MarceloRGonc
Copy link
Copy Markdown
Contributor

Fixes OPS-3884.

Copilot AI review requested due to automatic review settings March 10, 2026 15:58
@linear
Copy link
Copy Markdown

linear Bot commented Mar 10, 2026

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR (Fixes OPS-3884) aims to apply the newer route security policy configuration to flow-related API endpoints so they can be governed via project-scoped security policies.

Changes:

  • Added getProjectScopedRoutePolicy(...) to multiple flow endpoints to define project-scoped auth requirements.
  • Updated flow and flow-version controller route configs to use the new config.security policy shape (including permissions/principal types).
  • Minor formatting-only whitespace changes in the flow module.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.

File Description
packages/server/api/src/app/flows/flow/flow.controller.ts Adds project-scoped route security policies across multiple flow endpoints (create/update/list/count/get/delete/run).
packages/server/api/src/app/flows/flow/flow-version.controller.ts Adds project-scoped route security policy to the “update trigger config” endpoint and updates another request options block.
packages/server/api/src/app/flows/flow.module.ts Whitespace-only changes near websocket listener setup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread packages/server/api/src/app/flows/flow/flow.controller.ts
Comment thread packages/server/api/src/app/flows/flow/flow.controller.ts
Comment thread packages/server/api/src/app/flows/flow/flow-version.controller.ts
Comment thread packages/server/api/src/app/flows/flow.module.ts
@MarceloRGonc MarceloRGonc merged commit fd01e8f into main Mar 10, 2026
25 checks passed
@MarceloRGonc MarceloRGonc deleted the mg/flows branch March 10, 2026 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants