google / google/adk-python-community

Double Colon in Redis User Session ID Key

Ouverte
#34 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
Python
Étoiles
182
Forks
75
Métriques de merge des PR
Aucune PR mergée en 30 j

Description

## Description

The Redis session service implementation contains a bug where user session keys are generated with double colons ("::") in the key pattern. This occurs in the `RedisKeys.user_sessions()` method, which constructs keys used to track all sessions for a specific user within an application.

## Current Behavior

The key generation in `redis_session_service.py` currently produces:
```
{APP_PREFIX}::{app_name}:{user_id}
```

This results in keys like: `adk::my_app::user_123`

## Expected Behavior

The key should follow a consistent single-colon delimiter pattern:
```
{APP_PREFIX}{app_name}:{user_id}
```

Resulting in keys like: `adkmy_app:user_123`

## Root Cause

In the `RedisKeys.user_sessions()` method, the `APP_PREFIX` constant already contains a trailing colon, but the f-string adds an additional colon separator:

```python
return f"{State.APP_PREFIX}:{app_name}:{user_id}" # Creates double colon
```

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Ouvrez redis_session_service.py et examinez la méthode RedisKeys.user_sessions(), en commençant par la définition de APP_PREFIX et la construction actuelle de la clé. Vérifiez que la clé de session utilisateur générée utilise le modèle de délimiteurs attendu, puis consultez les tests de session Redis existants ou les sites d’appel de génération de clés pour vérifier le format obtenu.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python, redis
Domaine
backend, databases
Type d'issue
Bug
Difficulté
1/5
Temps estimé
Moins d'une heure
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.