envoyproxy/envoy

CacheFilter: Support customizing whether to cache responses with no cache-control directives

Open

#9 834 ouverte le 25 janv. 2020

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)C++ (5 373 forks)batch import
area/cachehelp wanted

Métriques du dépôt

Stars
 (27 997 stars)
Métriques de merge PR
 (Merge moyen 8j) (378 PRs mergées en 30 j)

Description

RFC7231§6.1 permits caching some responses to GET/HEAD requests without explicit cache-related headers, depending on the status code. To prevent unpleasant surprises, it should be configurable whether CacheFilter (#7198) caches such responses or whether it requires a positive indication of cacheability.

Guide contributeur

Direction de recherche
Cette demande propose d'ajouter une option configurable dans CacheFilter pour contrôler la mise en cache des réponses sans directives cache control explicites. L'implémentation doit modifier CacheFilter (probablement dans source/extensions/filters/http/cache/) pour ajouter un champ de configuration et ajuster la logique de mise en cache selon RFC7231. La PR existante #7198 a introduit CacheFilter ; le code associé peut se trouver dans source/extensions/filters/http/cache/config.h et cache filter.h. Les nouveaux contributeurs doivent étudier le modèle de configuration existant et la RFC de mise en cache HTTP.
Stack technique
c
Domaine
backend
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-2 jours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Récente
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
GitC++ basicsHTTP caching conceptsEnvoy proxy architecture
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
60