Skip to content

[Custom Engine Test] Test Pull Request - Custom Engine Safe Output#636

Closed
github-actions[bot] wants to merge 1 commit into
mainfrom
test-safe-outputs-custom-engine/d622c66350b2893f
Closed

[Custom Engine Test] Test Pull Request - Custom Engine Safe Output#636
github-actions[bot] wants to merge 1 commit into
mainfrom
test-safe-outputs-custom-engine/d622c66350b2893f

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

Test Pull Request - Custom Engine Safe Output

This pull request was automatically created by the test-safe-outputs-custom-engine workflow to validate the create-pull-request safe output functionality.

Changes Made

  • Created test file with timestamp
  • Demonstrates custom engine file creation capabilities

Test Information

  • Engine: Custom (GitHub Actions steps)
  • Workflow: test-safe-outputs-custom-engine
  • Trigger Event: push
  • Run ID: 17615499515

This PR can be merged or closed after verification of the safe output functionality.

Generated by Agentic Workflow Run

@github-actions github-actions Bot closed this Sep 10, 2025
@github-actions github-actions Bot deleted the test-safe-outputs-custom-engine/d622c66350b2893f branch September 10, 2025 14:07
github-actions Bot added a commit that referenced this pull request Jun 26, 2026
Add response Content-Type validation and a 50 MB size limit to the
network-to-file path in ensure-docs-slide-pdf.js, addressing the
js/http-to-file-access CodeQL alert (#636).

Changes:
- Reject responses whose Content-Type is not application/pdf or
  application/octet-stream before reading the body.
- Enforce a 50 MB ceiling via both the Content-Length header (early
  rejection) and the actual downloaded byte count (double-check).
- The existing PDF magic-bytes check (isPdf) is preserved as a
  third layer of validation.
- All new checks throw errors that are caught by the surrounding
  try/catch block, so the script still falls back to the placeholder
  PDF rather than crashing in sandboxed environments.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions Bot mentioned this pull request Jun 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants