allegro / allegro/envoy-control

Incoming permissions logs - handle data in invalid format

Aberta
#156 0 comentários 0 reações 0 responsáveis Ver no GitHub
Linguagem predominante
Kotlin
Estrelas
111
Forks
35
Métricas de merge de PRs
Nenhum PR com merge em 30d

Descrição

Currently we produce incoming permissions logs as a JSON, which is prepared in a non-reliable way. If particular characters (for example `"`) will be present in `x-service-name` header, the result JSON will be invalid.

To consider:
1. Fetch serviceName from client certificate SAN URI, instead of a header.
2. Change log format to something simpler than JSON, for example "every field in a new line"
3. Sanitize JSON strings

More information: https://github.com/allegro/envoy-control/pull/152#discussion_r468397992

Guia de contribuição

Nenhum guia de contribuição indexado para este repositório

Avaliação

Esta issue ainda não foi avaliada.

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.