envoyproxy/envoy

Route Matching during filter dispatch

Aberta

#6.824 aberto em 6 de mai. de 2019

Ver no GitHubGuia do colaborador
 (6 comentários) (2 reações) (0 responsável)C++ (5.373 forks)batch import
area/httpenhancementhelp wanted

Métricas do repositório

Stars
 (27.997 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

At present envoy does not support route matching to selectively dispatch a filter.

Nginx/Apache support configs like the following

<Location "/super-secret">
    AuthBasicProvider ldap
   AuthLDAPURL ldap://127.0.0.1/dc=example,dc=com?uid?one
    Require valid-user
</Location>

This config uses the ldap filter only when a route is matched. At present every filter must support an ad-hoc enable-disable flag. This should be supported directly.

Options:

  • Add RouteMatch to HTTPFilter
  • Standardize on request dynamic metadata key such as filter_name.enabled. This metadata can then be added as part of a route. httpConnectionManager should check this value before dispatching.

Guia do colaborador

Direção de pesquisa
Explore a lógica de despacho de filtros do gerenciador de conexão HTTP em source/common/http/filter/. Entenda como os filtros são atualmente habilitados/desabilitados. Em seguida, considere adicionar um campo RouteMatch ao proto HTTPFilter e modificar o gerenciador da cadeia de filtros para verificar a correspondência de rota antes do despacho.
Pilha de tecnologia
cpp
Domain
backendapi
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
3-5 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
35