envoyproxy/envoy

memory safety: audit use of static variables

Open

#11.130 geöffnet am 10. Mai 2020

Auf GitHub ansehen
 (5 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
help wantedtech debt

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

We should audit the use of static variables in the codebase to prevent static initialization/deinitialization order problems internally, and for dependent projects.

The codebase already defines a construct on first use macro. We should audit other static variables that should be migrated, and enforce the pattern via code linting.

Contributor Guide

Research-Richtung
Prüfen Sie alle statischen Variablen in der Codebasis, identifizieren Sie diejenigen, die das Konstrukt bei erster Verwendung Makro verwenden sollten, und erzwingen Sie dies durch Linting.
Tech Stack
cpp
Domain
backend
Issue Type
Refactoring
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-2 Tage
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
C++static variables
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
40