envoyproxy/envoy

CacheFilter: Parse and handle precondition request headers

Open

#9855 aperta il 28 gen 2020

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (5373 fork)batch import
area/cachehelp wanted

Metriche repository

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

Descrizione

CacheFilter (#7198) must fully parse and handle precondition request headers (RFC7232): if-match, if-none-match, if-modified-since, if-unmodified-since, and if-range.

This must be fixed before CacheFilter can be appropriate for production deployment.

We currently ignore these headers, which is incorrect, and will result in serving wrong content from cache. As a temporary shortcut, it would be acceptable to instead bypass caching for requests with precondition header. This would reduce cache effectiveness and give clients an easy way to bypass cache, but would never cause incorrect results. With the shortcut, this issue would no longer block production deployment of CacheFilter.

Guida contributor

Direzione di ricerca
Analizza l'implementazione corrente di CacheFilter in source/extensions/filters/http/cache/ per capire dove avviene il parsing degli header. Gli header di precondizione (if match, if none match, ecc.) devono essere analizzati e utilizzati per decidere gli hit/miss della cache. Come scorciatoia temporanea, considera di aggiungere una logica per saltare la memorizzazione nella cache se è presente uno qualsiasi di questi header. Il problema fa riferimento a RFC 7232 per la specifica.
Tech stack
Nessuno
Dominio
backend
Tipo issue
Bug
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.
Abbastanza chiara
Prerequisiti
C++HTTP
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65