envoyproxy / envoyproxy/nighthawk

Test-server per-request configuration: add an option to clear values

Open
#83 0 comments 0 reactions 0 assignees View on GitHub
good first issue help wanted
Dominant language
C++
Stars
414
Forks
95
Avg merge
2d 9h
Merged PRs (30d)
11

Description

Currently the test server will apply per-request configuration specified in per request-header to the process-level configuration, by performing a proto-level `Merge()`.
This doesn't always work well: it's not possible to override the server configuration with type-specific defaults. For example, response-size (int, default=0) cannot be overridden to 0 when the server-level configuration is non-zero.
A bool-valued field called `clear` could be helpful here, to allow the client to indicate it wants to fully specify the configuration (and not inherit from what the server has configured).

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.