cleveragents/cleveragents-core
4
0
Fork 3
Code Issues 6.0k Pull Requests 115 Actions Packages Projects Releases Wiki Activity

fix(a2a): validate session_id at entry of _handle_session_close before devcontainer cleanup #11089

Closed
HAL9000 wants to merge 1 commits from pr-fix-#11053-session-id-validation into master
pull from: pr-fix-#11053-session-id-validation
merge into: cleveragents:master
Conversation 7 Commits 1 Files Changed 5
+32 -25

1 Commits

Author SHA1 Message Date
HAL9000 4ac4d3608a fix(a2a): validate session_id at entry of _handle_session_close before devcontainer cleanup
CI / helm (pull_request) Successful in 40s
Details
CI / build (pull_request) Successful in 1m12s
Details
CI / lint (pull_request) Successful in 1m43s
Details
CI / benchmark-publish (pull_request) Has been skipped
Details
CI / quality (pull_request) Successful in 1m54s
Details
CI / typecheck (pull_request) Successful in 2m7s
Details
CI / push-validation (pull_request) Successful in 34s
Details
CI / security (pull_request) Successful in 2m13s
Details
CI / benchmark-regression (pull_request) Failing after 1m8s
Details
CI / e2e_tests (pull_request) Successful in 4m35s
Details
CI / integration_tests (pull_request) Failing after 4m40s
Details
CI / unit_tests (pull_request) Failing after 5m14s
Details
CI / docker (pull_request) Has been skipped
Details
CI / coverage (pull_request) Has been skipped
Details
CI / status-check (pull_request) Failing after 2s
Details 
Move session_id validation to the top of _handle_session_close() in
A2aLocalFacade so that empty or missing session IDs are rejected with
ValueError('session_id is required') regardless of whether SessionService
is wired. Previously, callers could bypass the guard by omitting
session_service and passing an empty/null session ID through to
_cleanup_session_devcontainers().

Also close out issue #9094.

ISSUES CLOSED: #11053
 2026-05-12 20:48:50 +00:00