envoyproxy/envoy

accesslog: expose all fields in FileAccessLogger in data.accesslog.v3.AccessLog for ALS

Open

#31.636 aberto em 4 de jan. de 2024

Ver no GitHub
 (4 comments) (0 reactions) (0 assignees)C++ (5.373 forks)batch import
area/access_logenhancementhelp wanted

Métricas do repositório

Stars
 (27.997 stars)
Métricas de merge de PR
 (Mesclagem média 8d) (378 fundiu PRs em 30d)

Description

Title: expose all fields in FileAccessLogger into data.accesslog.v3.AccessLog for ALS

Description:

There are multiple fields availiable in FileAccessLogger, but some of them were not exposed in ALS, considering ALS have better performance compare with FileAccessLogger to collect access logs, expose those fields in ALS is what needed.

e.g. DOWNSTREAM_PEER_ISSUER, UPSTREAM_PEER_CERT, DOWNSTREAM_PEER_FINGERPRINT_256, DOWNSTREAM_PEER_FINGERPRINT_1, DOWNSTREAM_PEER_SERIAL, DOWNSTREAM_HANDSHAKE_DURATION etc...

Guia do colaborador

Direção de pesquisa
Revise o código fonte do FileAccessLogger e o proto AccessLog, depois adicione os campos ausentes e implemente o preenchimento.
Pilha de tecnologia
cpp
Domain
backendobservability
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
Meio dia
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
GitC++Protocol Buffers
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
65