envoyproxy/envoy

Allow setting a destination for log messages from wasm plugins

Open

#22.669 aperta il 12 ago 2022

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

Metriche repository

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

Descrizione

Title: Allow setting a destination for log messages from wasm plugins

Description:

Currently, logging from wasm plugins all go to envoy's app logs. However, there are use cases for separating out log files for specific domains - for example, a WAF filter will often need to have a separate audit log which is used for actioning on suspicious events.

One approach could be for the config to define log destinations

message LogDestination {
  oneof {
    string file_path = 1;
    
    string remote_log_service = 2;
  }
}

message VmConfig {
  ...
  map<string, LogDestination> log_destination

And the log ABI would be extended to accept a log destination string which would be mapped with that config.

[optional Relevant Links:]

https://github.com/envoyproxy/envoy/issues/22557#issuecomment-1205785100

Guida contributor

Direzione di ricerca
Comprendi l'attuale meccanismo di logging per i plugin WASM in Envoy (source/extensions/wasm/config.cc e file correlati). Quindi estendi il protobuf VmConfig per includere le destinazioni dei log e modifica le funzioni ABI (proxy log) per accettare un nome di destinazione, mappandolo alla destinazione configurata. Coordinati con l'infrastruttura di logging esistente.
Tech stack
cpp
Dominio
backendapi
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
C++Envoy development setupWASM plugin concepts
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50