envoyproxy/envoy

Unify URL parsing with GoogleURL

Open

#15.290 aperta il 3 mar 2021

Vedi su GitHub
 (1 commento) (1 reazione) (1 assegnatario)C++ (5373 fork)batch import
area/httparea/securityhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

Two concerns:

  • llhttp (http-parser replacement) does not have a URL parser: https://github.com/nodejs/llhttp/issues/7. Derek brought up this concern in that issue.
  • The fewer different URL parsing stuff we have, the better.

Envoy uses http_parser_url to split the URL into pieces.

We should replace this with GoogleURL, which does URL parsing (https://chromium.googlesource.com/chromium/src/+/master/url#parsing). See https://chromium.googlesource.com/chromium/src/+/master/url/gurl.h#340

Related Issues: https://github.com/envoyproxy/envoy/issues/5155 https://github.com/envoyproxy/envoy/issues/6588

Can we bring back https://github.com/envoyproxy/envoy/pull/11670?

cc @derekargueta @dio @envoyproxy/security-team

Guida contributor

Direzione di ricerca
Analizza la precedente PR #11670 e le issue correlate #5155 e #6588 per comprendere i tentativi precedenti. Rivedi l'uso attuale di http parser url in source/common/http/utility.cc e l'API della libreria GoogleURL su https://chromium.googlesource.com/chromium/src/+/master/url/gurl.h. Determina come sostituire le chiamate http parser url con il parsing GURL, assicurandoti che tutti i campi dell'URL vengano estratti correttamente.
Tech stack
cpp
Dominio
backendinfrastructure
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-2 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++GitCMake
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
40