Skip to content

Add overview for body size limit configuration#830

Open
ayaoba24 wants to merge 2 commits into
Streampay-Org:mainfrom
ayaoba24:OpenTelemetry-tracing
Open

Add overview for body size limit configuration#830
ayaoba24 wants to merge 2 commits into
Streampay-Org:mainfrom
ayaoba24:OpenTelemetry-tracing

Conversation

@ayaoba24

Copy link
Copy Markdown

Added overview section for body size limit configuration.

closes #577

Security Changes

Type of Security Change

  • SAST rule update
  • Dependency vulnerability fix
  • Exemption addition/renewal
  • Security workflow modification
  • Container image update
  • Other: _______________

Vulnerability Details (if applicable)

CVE/Advisory ID:

  • CVE-ID:
  • GHSA-ID:

Affected Package:

  • Name:
  • Version:
  • Severity: [ ] Critical [ ] High [ ] Medium [ ] Low

Fix Applied:

  • Package version bump
  • Code change to mitigate
  • Configuration update
  • Exemption granted (see below)

Exemption Request (if applicable)

Exemption ID: EXEMPT-___

Justification:

Mitigation Applied:

Expiry Date: YYYY-MM-DD (max 90 days from now)

Review Plan:

Testing

  • Ran npm audit locally - output attached or no new vulnerabilities
  • Security workflow passes on this branch
  • Test suite passes: npm test
  • Build succeeds: npm run build

Security Impact Analysis

Affected Components:

  • Authentication/Authorization
  • Payment processing
  • Data encryption
  • API endpoints
  • Dependencies
  • Container images
  • CI/CD pipeline
  • Other: _______________

Risk Assessment:

Documentation Updates

  • Updated README.md (if workflow changed)
  • Updated SECURITY-CI-SETUP.md (if process changed)
  • Updated security-exemptions.json (if applicable)
  • Added security notes to code comments

Checklist

  • No secrets or keys committed
  • No PII or sensitive data in logs
  • All security scans pass (or exemptions documented)
  • Branch protection requirements met
  • Code review from security team (for critical changes)

Additional Notes

Test Output

# Paste npm test output here
npm test

# Paste npm audit output here (if relevant)
npm audit

CI Run Link

Workflow Run:


Security Review Required: @security-team
Compliance Impact: [Yes/No - explain if yes]

ayaoba24 and others added 2 commits June 30, 2026 10:43
@drips-wave

drips-wave Bot commented Jul 2, 2026

Copy link
Copy Markdown

@ayaoba24 Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@ayaoba24

ayaoba24 commented Jul 2, 2026

Copy link
Copy Markdown
Author

please review and merge my pr

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.

Add OpenTelemetry tracing across API + lib

1 participant