envoyproxy/envoy
Ver no GitHub[dataplane] reset ActiveStream::streamInfo().route() when clearRouteCache()
Open
#20.157 aberto em 1 de mar. de 2022
area/httpbughelp wanted
Métricas do repositório
- Stars
- (27.997 stars)
- Métricas de merge de PR
- (Mesclagem média 8d) (378 fundiu PRs em 30d)
Description
Title: reset ActiveStream::streamInfo().route() when clearRouteCache()
Description: There is a gap between the ActiveStreamFilterBase::clearRouteCache and ActiveStreamFilterBase::streamInfo()::route():
whenever we call clearRouteCache(), the {en,de}coder_callback_.route() ONLY!, the streamInfo().route() is NOT updated.
The "new" route() on {en,de}coder_callback_ will be reset when we refreshCachedRoute(), by either explicitly calling it, or calling {en,de}coder_callback_.route().
So after a clearCachedRoute(), the route() on streamInfo is "stale" and will be out-of-date until refreshCachedRoute() is called.
we should explicitly clear/reset streamINfo()::route_ in clearRouteCache()