Skip to content

fuzz: add router fuzz proto#12727

Merged
asraa merged 3 commits intoenvoyproxy:masterfrom
samflattery:router_fuzz_proto
Aug 21, 2020
Merged

fuzz: add router fuzz proto#12727
asraa merged 3 commits intoenvoyproxy:masterfrom
samflattery:router_fuzz_proto

Conversation

@samflattery
Copy link
Contributor

Commit Message: Add router fuzz proto
Additional Description:

  • I'm working on a router fuzzer with @asraa and another intern may be taking over the task after I leave
  • We laid out a plan for the fuzzer here [fuzz] Router Fuzzing  #12568
  • this is the proposed input for the fuzzer, open to feedback

Risk Level: Low
Testing: N/A
Docs Changes: N/A
Release Notes: N/A

Sam Flattery added 2 commits August 19, 2020 14:44
Signed-off-by: Sam Flattery <samflattery@google.com>
Signed-off-by: Sam Flattery <samflattery@google.com>
@samflattery
Copy link
Contributor Author

/cc @asraa

@asraa asraa self-assigned this Aug 19, 2020
Copy link
Contributor

@asraa asraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. What do you think about adding a config field to fuzz the router configuration? envoy::extensions::filters::http::router::v3::Router& config. The change would be relatively simple since FilterConfig has a constructor that takes this proto, so the test library would just have two constructors.

Signed-off-by: Sam Flattery <samflattery@google.com>
@samflattery
Copy link
Contributor Author

Good idea, done!

Copy link
Contributor

@asraa asraa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@asraa asraa merged commit c003363 into envoyproxy:master Aug 21, 2020
lavignes added a commit to lavignes/envoy that referenced this pull request Aug 24, 2020
* envoy/master: (90 commits)
  cleanup: use structured binding (envoyproxy#12791)
  docs: fix header name for retries in gRPC services (envoyproxy#12790)
  docs: clarify meaning of HeaderValueOption.append (envoyproxy#12792)
  doc: clarify handling of duplicate xDS resource names (envoyproxy#12756)
  Dependencies: build updates. (envoyproxy#12786)
  Ratelimit: Add optional descriptor key to generic_key action (envoyproxy#12734)
  test: refactor header inclusion to speed up building (for test/mocks/upstream:upstream_mocks)  (envoyproxy#12407)
  docs: Fix omitted word (envoyproxy#12782)
  ci: avoid uploading dwp as separate artifact (envoyproxy#12777)
  doc: Fix small typos (envoyproxy#12769)
  fix cache factory category (envoyproxy#12765)
  docs: fix typo v1.15.0.rst (envoyproxy#12680)
  Add clang-cl RBE toolchain for Windows (envoyproxy#12776)
  fuzz: add router fuzz proto (envoyproxy#12727)
  header: New HeaderMatcher and StringMatcher type - Contains (envoyproxy#12623)
  tcp_proxy: use dynamicMetadata() from StreamInfo for load balancing (envoyproxy#12595)
  network: add io handle recv function for http inspector (envoyproxy#12736)
  jwt_authn: supports jwt payload without "iss" field (envoyproxy#12744)
  Add support for nested JSON format in json logging mode (envoyproxy#12602)
  http: fixing a fuzz flake by setting details on connection teardown (envoyproxy#12737)
  ...

Signed-off-by: Scott LaVigne <lavignes@amazon.com>
@samflattery samflattery deleted the router_fuzz_proto branch August 31, 2020 11:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants