Skip to content

Honor filesystem permissions for app file uploads - #38416

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/63b239c5066dcb9d18b5050089c659fb2a784909
Aug 13, 2026
Merged

Honor filesystem permissions for app file uploads#38416
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/63b239c5066dcb9d18b5050089c659fb2a784909

Conversation

@copyberry

@copyberry copyberry Bot commented Aug 13, 2026

Copy link
Copy Markdown

Honor filesystem permissions for app file uploads

Why

App tool file arguments could read and upload files without applying the active
filesystem sandbox policy.

What changed

  • Apply the effective filesystem policy, including session and turn grants, when
    reading metadata and streaming files for upload.
  • Reject uploads before contacting the files endpoint when the selected path is
    denied.
  • Report an error when the executor cannot provide sandboxed file streaming for
    a restricted policy.

Testing

Added integration coverage showing that restricted policies allow permitted
files and reject denied files before upload.

## Why

App tool file arguments could read and upload files without applying the active
filesystem sandbox policy.

## What changed

- Apply the effective filesystem policy, including session and turn grants, when
  reading metadata and streaming files for upload.
- Reject uploads before contacting the files endpoint when the selected path is
  denied.
- Report an error when the executor cannot provide sandboxed file streaming for
  a restricted policy.

## Testing

Added integration coverage showing that restricted policies allow permitted
files and reject denied files before upload.

GitOrigin-RevId: 63b239c5066dcb9d18b5050089c659fb2a784909
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/63b239c5066dcb9d18b5050089c659fb2a784909 branch from ea7a2a9 to 4f70321 Compare August 13, 2026 18:37
@copyberry
copyberry Bot merged commit 4f70321 into main Aug 13, 2026
27 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/63b239c5066dcb9d18b5050089c659fb2a784909 branch August 13, 2026 18:38
@github-actions github-actions Bot locked and limited conversation to collaborators Aug 13, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants