Repository navigation
Commit e44692c
committed
fix(sessions): list only the empty user's sessions in FirestoreSessionService
list_sessions checked `if user_id:`, so user_id="" fell through to the
all-users collection_group query and returned every user's sessions.
Check `is not None` instead, as the other session services do.
Fixes #76621 parent d324744 commit e44692c
2 files changed
Lines changed: 28 additions & 2 deletions
File tree
- src/google/adk/integrations/firestore
- tests/unittests/integrations/firestore
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
368 | 368 | | |
369 | 369 | | |
370 | 370 | | |
371 | | - | |
| 371 | + | |
372 | 372 | | |
373 | 373 | | |
374 | 374 | | |
| |||
392 | 392 | | |
393 | 393 | | |
394 | 394 | | |
395 | | - | |
| 395 | + | |
396 | 396 | | |
397 | 397 | | |
398 | 398 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
961 | 961 | | |
962 | 962 | | |
963 | 963 | | |
| 964 | + | |
| 965 | + | |
| 966 | + | |
| 967 | + | |
| 968 | + | |
| 969 | + | |
| 970 | + | |
| 971 | + | |
| 972 | + | |
| 973 | + | |
| 974 | + | |
| 975 | + | |
| 976 | + | |
| 977 | + | |
| 978 | + | |
| 979 | + | |
| 980 | + | |
| 981 | + | |
| 982 | + | |
| 983 | + | |
| 984 | + | |
| 985 | + | |
| 986 | + | |
| 987 | + | |
| 988 | + | |
| 989 | + | |
964 | 990 | | |
965 | 991 | | |
966 | 992 | | |
| |||
0 commit comments