Skip to content
This repository was archived by the owner on Aug 17, 2026. It is now read-only.

fix: allow session-provenance canvas reads outside cwd (#134) - #143

Merged
joeshull merged 6 commits into
mainfrom
team/canvas-outside-cwd-containment
Aug 14, 2026
Merged

fix: allow session-provenance canvas reads outside cwd (#134)#143
joeshull merged 6 commits into
mainfrom
team/canvas-outside-cwd-containment

Conversation

@joeshull

Copy link
Copy Markdown

Summary

  • Session-scoped provenance allow-list for write/edit/ast_edit/canvas absolute paths
  • /api/file* serves those paths; /etc/passwd and .. stay 403
  • canvas() may name absolute deliverable paths

Test plan

Closes #134

@joeshull
joeshull force-pushed the team/canvas-outside-cwd-containment branch 2 times, most recently from 13728c9 to c420160 Compare August 14, 2026 00:08
@joeshull
joeshull enabled auto-merge (squash) August 14, 2026 00:09
@joeshull
joeshull force-pushed the team/canvas-outside-cwd-containment branch 2 times, most recently from b20bc90 to c25b71a Compare August 14, 2026 00:30
@joeshull
joeshull force-pushed the team/canvas-outside-cwd-containment branch from c25b71a to b6b07ef Compare August 14, 2026 00:41
@joeshull
joeshull merged commit bb4fffd into main Aug 14, 2026
4 checks passed
@joeshull
joeshull deleted the team/canvas-outside-cwd-containment branch August 14, 2026 00:43
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.

Canvas fails to load files outside session cwd ("path outside working directory")

1 participant