envoyproxy/envoy

wasm: metrics never expire and accumulate

Open

#14 070 ouverte le 17 nov. 2020

Voir sur GitHub
 (14 commentaires) (8 réactions) (0 assignés)C++ (5 373 forks)batch import
area/statsarea/wasmbughelp 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

Wasm filter implements the stats by allocating Envoy stats in the context and never de-allocating them. Since ABI encourages dynamic values, the stats accumulate and eventually leak (also, potentially exposing a security risk if stat values are untrusted).

Guide contributeur

Direction de recherche
Étudiez l'implémentation du filtre Wasm dans le code source d'Envoy (probablement sous source/extensions/filters/http/wasm/). Comprenez comment les statistiques d'Envoy sont actuellement allouées via l'ABI et pourquoi elles n'expirent jamais. Examinez la discussion dans l'issue #14070 pour les solutions proposées et les retours des mainteneurs. Envisagez de mettre en œuvre un mécanisme pour suivre et désallouer les statistiques lorsque le module Wasm est détruit ou via un nettoyage périodique.
Stack technique
cppwasm
Domaine
backendperformancesecurityobservability
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.
4
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
3-5 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Ancienne
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Envoy proxy basicsWebAssembly modulesEnvoy stat subsystem
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
25