Skip to content

ci(security): integrate Bright CI pipeline for security tests and remediation#848

Open
bright-security-golf[bot] wants to merge 18 commits intostablefrom
bright/fbde6154-3777-403b-b436-d97063c665b0
Open

ci(security): integrate Bright CI pipeline for security tests and remediation#848
bright-security-golf[bot] wants to merge 18 commits intostablefrom
bright/fbde6154-3777-403b-b436-d97063c665b0

Conversation

@bright-security-golf
Copy link

@bright-security-golf bright-security-golf bot commented Jan 23, 2026

Note

Fixed 13 of 14 vulnerabilities.
Please review the fixes before merging.

Fix Vulnerability Endpoint Affected Files Resolution
[Critical] Server Side Template Injection POST /api/render src/app.controller.ts Sanitize user input by removing template expressions before rendering to prevent injection.
[Critical] XPATH Injection GET /api/partners/partnerLogin src/partners/partners.service.ts, src/partners/partners.controller.ts Sanitize user inputs for XPath queries to prevent injection attacks by replacing single quotes with XML-safe entities.
[Critical] XPATH Injection GET /api/partners/searchPartners src/partners/partners.service.ts Sanitize and validate XPath input to prevent injection attacks by allowing only safe characters.
[Critical] SQL Injection POST /graphql src/products/products.resolver.ts, src/products/products.service.ts Replaced dynamic SQL query with a parameterized query using MikroORM's query builder to prevent SQL injection.
[High] Server Side Request Forgery GET /api/file/azure src/file/file.controller.ts Added validation to ensure paths start with '/' to prevent SSRF attacks.
[High] Server Side Request Forgery GET /api/file/google src/file/file.controller.ts Added validation to ensure the path starts with a forward slash to prevent SSRF attacks.
[High] Server Side Request Forgery GET /api/file/digital_ocean src/file/file.controller.ts Added validation to ensure the 'path' parameter starts with a forward slash to prevent SSRF attacks.
[High] Server Side Request Forgery GET /api/file/aws src/file/file.controller.ts, src/file/cloud.providers.metadata.ts Added validation to ensure the path starts with a forward slash and restricted provider URLs to known values.
[High] [BL] ID Enumeration GET /api/users/id/1 src/users/users.controller.ts Added authorization check to ensure users can only access their own information by verifying the requester's identity against the requested user ID.
[Medium] GraphQL Introspection POST /graphql src/app.module.ts Disabled GraphQL introspection in the server configuration to prevent schema exposure.
[Medium] Secret Tokens Leak GET /api/secrets src/app.controller.ts Replaced hardcoded secret tokens with environment variables to prevent leaks.
[Medium] GraphQL Introspection POST /graphql src/app.module.ts Enhanced GraphQL configuration to prevent introspection and customize error messages.
[Medium] GraphQL Introspection POST /graphql src/app.module.ts Added a custom validation rule to block introspection queries by checking for fields starting with '__'.
[Medium] [BL] Business Constraint Bypass GET /api/products/latest src/products/products.controller.ts, src/products/products.service.ts Attempted fix: Ensure the 'limit' parameter is validated as a positive integer and enforce a maximum limit in the controller to prevent bypass.
Workflow execution details
  • Repository Analysis: JavaScript, TypeScript, NestJS
  • Entrypoints Discovery: 59 entrypoints found
  • Attack Vectors Identification
  • E2E Security Tests Generation: 59 test files generated
  • E2E Security Tests Execution: 14 vulnerabilities found
  • Cleanup Irrelevant Test Files: 45 test files removed
  • Applying Security Fixes: 14 fixes generated
  • E2E Security Tests Execution: 3 vulnerabilities found
  • Cleanup Irrelevant Test Files: 10 test files removed
  • Applying Security Fixes: 14 fixes generated
  • E2E Security Tests Execution: 1 vulnerabilities found
  • Cleanup Irrelevant Test Files: 3 test files removed
  • Applying Security Fixes: 14 fixes generated
  • E2E Security Tests Execution: 1 vulnerabilities found
  • Cleanup Irrelevant Test Files: 0 test files removed
  • Applying Security Fixes: 14 fixes generated
  • E2E Security Tests Execution: 1 vulnerabilities found
  • Cleanup Irrelevant Test Files: 0 test files removed
  • Applying Security Fixes: 14 fixes generated
  • Workflow Wrap-Up

@bright-security-golf bright-security-golf bot force-pushed the bright/fbde6154-3777-403b-b436-d97063c665b0 branch from 591e621 to fc5903b Compare January 23, 2026 21:27
@bright-security-golf bright-security-golf bot force-pushed the bright/fbde6154-3777-403b-b436-d97063c665b0 branch from b355fbd to 78986b7 Compare January 23, 2026 21:37
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.

0 participants

Comments