envoyproxy/envoy

Stateful Session cookies incompatible between Envoy Processes.

Open

#44 111 ouverte le 25 mars 2026

Voir sur GitHub
 (1 commentaire) (0 réactions) (1 assigné)C++ (5 373 forks)batch import
bughelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 7j 21h) (260 PRs mergées en 30 j)

Description

Title: Stateful Session cookies incompatible between Envoy Processes.

Description: When using Envoy as an Edge Proxy with multiple instances (an NLB in front) AND using the StatefulSession with cookies doesn't work properly.

The cookie is encoded with the expiry time BUT it uses the monotonic clock:

https://github.com/envoyproxy/envoy/blob/21b70d126c0a5c16d844658ee2db68f94deb7548/source/extensions/http/stateful_session/cookie/cookie.cc#L21-L22

When HTTP requests are routed to different Envoy processes, they will have different values for their monotonic clock and the cookie can inadvertently expire sooner then its should, which will cause a new backend to get selected.

I believe the appropriate fix is for time_source_.systemTime() to be used so that two envoy processes can validate each other cookies.

cc @cpakulski

Guide contributeur

Direction de recherche
Examinez le codage des cookies dans source/extensions/http/stateful session/cookie/cookie.cc aux lignes 21-22, où l'horloge monotone est utilisée pour l'expiration. Proposez de passer à systemTime() pour garantir la cohérence entre les processus. Comprenez l'impact sur la persistance de session sous des instances Envoy équilibrées.
Stack technique
cpp
Domaine
backendapi
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
C++Envoy development
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50