envoyproxy/envoy

Move HCM user agent stats functionality into an optional filter

Open

#10.195 geöffnet am 27. Feb. 2020

Auf GitHub ansehen
 (11 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)C++ (5.373 Forks)batch import
area/httpbeginnerhelp wanted

Repository-Metriken

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

Beschreibung

Right now we have user agent detection built directly into the HCM: https://github.com/envoyproxy/envoy/blob/master/source/common/http/user_agent.h

This code is very old and at this point should not be required to be built into the HCM. We should move this code into an optional filter.

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandenen Dateien user agent.h und user agent.cc in source/common/http, um die aktuelle Implementierung zu verstehen. Erstellen Sie dann einen neuen Filter, der dieselbe Funktionalität als optionalen Filter bereitstellt, und folgen Sie dem Muster anderer Filter im Repository. Stellen Sie sicher, dass er über die Konfiguration aktiviert/deaktiviert werden kann.
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
GitC++Envoy development basics
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65