Skip to content

fix: validate temp session IDs before loading existing sessions#11

Open
luojiyin1987 wants to merge 2 commits intoApeCodeAI:masterfrom
luojiyin1987:fix/session-id-validation
Open

fix: validate temp session IDs before loading existing sessions#11
luojiyin1987 wants to merge 2 commits intoApeCodeAI:masterfrom
luojiyin1987:fix/session-id-validation

Conversation

@luojiyin1987
Copy link
Copy Markdown
Contributor

Summary

  • separate temp session creation from loading an existing session
  • validate session_id before using it in filesystem paths
  • add regression coverage for missing sessions and path traversal style IDs

Testing

  • uv run pytest -q tests/test_temp_manager.py
  • uv run pytest -q

Closes #10

@vercel
Copy link
Copy Markdown

vercel bot commented Mar 26, 2026

@luojiyin1987 is attempting to deploy a commit to the apecodelab's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Fix temp session lookup creating directories for invalid session IDs

1 participant