df52004032
[iwyu] Add test/core/end2end ( #29998 )
...
* [iwyu] Add test/core/end2end
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* fix
* fix
* fix
* update
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
64939531f9
Re-Reland: Add SRV and TXT record lookup methods to the iomgr API ( #30246 )
...
* Revert "Revert "Reland: Add SRV and TXT record lookup methods to the iomgr PAI (#30242 )"
This reverts commit b5966f39eb
2 years ago
d7e6878ec4
[chttp2] use new channel args type ( #30252 )
...
* x
* x
* Automated change: Fix sanity tests
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
b5966f39eb
Revert "Reland: Add SRV and TXT record lookup methods to the iomgr PAI ( #30242 )
...
* Revert "Reland: Add SRV and TXT record lookup methods to the iomgr API (#30206 )"
This reverts commit c229703f9f
2 years ago
d304712f64
[channel_args] Spread grpc_core::ChannelArgs through client channel code ( #30008 )
...
* [channel_args] Spread grpc_core::ChannelArgs through client channel code
* progress
* progress
* grpc compiles
* uniqueptr+compiles
* fix
* fix
* fix
* fix
* fix
* fix
* fixes
* fix
* fix
* fixes
* fix-lb
* fix
* fix up arg construction
* fix
* fix
* fix
* fix
* move to const& to save bytes
* fix
* fix
* progress
* review feedback
* fix
* fix
* fixes
* fixes
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* debug-helper
* Automated change: Fix sanity tests
* fix
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
2 years ago
c229703f9f
Reland: Add SRV and TXT record lookup methods to the iomgr API ( #30206 )
...
* Revert "Revert "Add SRV and TXT record lookup methods to the iomgr API (#30078 )" (#30176 )"
This reverts commit 2c3acbb2b20fce0ae150
2 years ago
2c3acbb2b2
Revert "Add SRV and TXT record lookup methods to the iomgr API ( #30078 )" ( #30176 )
...
This reverts commit c835402dd9
2 years ago
c835402dd9
Add SRV and TXT record lookup methods to the iomgr API ( #30078 )
...
* Rename ResolveName to LookupHostname (same as EventEngine)
* Add stubs and no-op impls for LookupTXT and LookupSRV
* add native resolver tests that assert unimplemented
* extract custom name_server-setting logic and remove goto
* Separate SRV queries from grpc_dns_lookup_ares
* add necessary fixits before merging
* Automated change: Fix sanity tests
* fix missing ExecCtx on resolver tests
* separate out TXT lookup from hostname lookup (now all 3 are separate)
* rm DNS and update docs
* fix the fixer (forgot to add deps to BUILD)
* remove unused SRV and TXT args from ares hostname lookup method
* rename hostname-only ares dns lookup method
* refactor AresRequest using template method pattern
* Add name_server to Ares LookupHostname internals (needs iomgr API change)
* fix resolver test, callback should not be called on cancellation
* implement Ares-iomgr SRV and TXT lookup methods (verified manually)
Used a custom bind server with some redacted tests from
`resolve_address_test` to ensure both are working as expected.
* cleanup cruft
* unify common ares request setup logic between A, AAAA, TXT, and SRV
* generate_projects
* comment out unused args
* DNSResolver iomgr API uses Duration; hostname has all args now
* rm stale TODOs
* windows fix - bad variable name
* windows fix
* Automated change: Fix sanity tests
* reviewer feedback
* make protected members private
* move common properties to AresRequest base class
* localhost TXT results are empty, not an error
* reviewer feedback
* fix
Co-authored-by: drfloob <drfloob@users.noreply.github.com>
2 years ago
8249bb97e2
[event_engine] Fix initialization ( #30107 )
...
* [event_engine] Fix initialization
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
7cbffb9218
[mac] Disable API fuzzer ( #30105 )
...
* mac fix
* Automated change: Fix sanity tests
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
24caf53835
Roll forward iomgr ee fork ( #30022 )
...
* Revert "Revert "[event-engine] Fork timer code (#29816 )" (#30020 )"
This reverts commit cda2127776
3 years ago
f8eedac1fc
Changing grpc_tcp_client_vtable to include TCP cancel connect method ( #29968 )
...
* Changing grpc_tcp_client_vtable to include TCP cancel connect method
* fix unused parameter error
* update
* fix sanity checks
3 years ago
c6ebbb5482
Added GRPC_ERROR_IS_NONE ( #29937 )
...
* Added GRPC_ERROR_IS_NONE
* Convert
* Fix
3 years ago
50ae54dd1c
[fuzzing] Run fuzzers a little bit as a test ( #29919 )
...
* [fuzzer] Add a script to sample fuzzers
* remember the script
* add ci
* bleh
* fix
* Update sample_fuzzers.sh
* tweak
* tweak
* tweak
* tweak
* tweak
* fix fuzzer found bug
* add explainer
* make it bold af
* limit max fuzzing time in addition to runs
3 years ago
6e2d7b984d
[chttp2] Fix use-after-free ( #29916 )
3 years ago
c11f66faef
Reland DNSResolver cancellation ( #29581 and #29631 ) ( #29670 )
...
* Reland DNSResolver cancellation (#29581 and #29631 )
* add failing test: pollset_set deletion after DNSResolver cancellation
* Allow interested_parties to make like a pumpkin after cancellation
* cleanup
* iwyu
* Automated change: Fix sanity tests
* get the pollset_set join correct
* reviewer feedback
Co-authored-by: drfloob <drfloob@users.noreply.github.com>
3 years ago
9f7311e399
Revert DNSResolver cancellation ( #29581 and #29631 ) ( #29634 )
...
This reverts commit 075e84314d
3 years ago
0a21ecdff6
Change cancellation semantics in the iomgr DNSResolver ( #29581 )
...
DNS requests were previously cancellable, but it was assumed
that the resolution callback would be called in all cases. Now, requests
provide a `bool Cancel(TaskHandle handle)` method that lets callers know
whether cancellation is successful, and if so, the callback will not be
run.
This is in accordance with EventEngine's cancellation semantics, and it
is a step towards migration from iomgr to EventEngine.
3 years ago
8bb45aa3a6
Convert grpc_channel to C++ ( #29266 )
...
* begin
* tests
* fix
* http
* Filter fuzzer
* progress
* basics
* progress
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f6724d02d2e7307960842f48a6f224e4a1951844e8574d641c42814001b957cbd135c61043
3 years ago
371d231b53
Support unix socket in grpc_sockaddr_to_string ( #29169 )
...
* Support unix socket in grpc_sockaddr_to_string
* make it return statusor
* clang fix
* made grpc_sockaddr_to_string() to return statusor
* Let Chttp2ServerListener::Start crash
* test failure fixed
* api_fuzzer fixed
* comments addressed.
* more comments addressed
* comments addressed
* fix other broken builds
3 years ago
33aaa55032
Filter fuzzer ( #29177 )
...
* begin
* tests
* fix
* http
* Filter fuzzer
* progress
* basics
* progress
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* Revert "Revert "HTTP Client Filter --> promises (#29031 )" (#29181 )"
This reverts commit 6ee276f6724d02d2e7307960842f48a6f224e4a1951844e8574d641c42814001b957cbd135c61043
3 years ago
18a8f6aad9
Refactor end2end tests to exercise each EventEngine ( #29202 )
...
* Refactor end2end tests to exercise each EventEngine
* fix incorrect bazel_only exclusions
* Automated change: Fix sanity tests
* microbenchmark fix
* sanitize, fix iOS flub
* Automated change: Fix sanity tests
* iOS fix
* reviewer feedback
* first pass at excluding EventEngine test expansion
Also caught a few cases where we should not test pollers, but should
test all engines. And two cases where we likely shouldn't be testing
either product.
* end2end fuzzers to be fuzzed differently via EventEngine.
* sanitize
* reviewer feedback
* remove misleading comment
* reviewer feedback: comments
* EE test_init needs to play with our build system
* fix golden file test runner
Co-authored-by: drfloob <drfloob@users.noreply.github.com>
3 years ago
5fc3ff8203
grpc_millis -> Timestamp/Duration ( #28119 )
...
* wip
* Automated change: Fix sanity tests
* fixes
* progress
* progress
* grpc compiles
* Automated change: Fix sanity tests
* fixing tests
* x
* progress
* better code
* Automated change: Fix sanity tests
* progress
* progress
* windows fix
* Make Duration metadata trivial
* better message
* fix
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* progress
* fixes
* fix
* fix
* spam
* un-disable errantly disabled tests
* gain insight
* Automated change: Fix sanity tests
* fixes
* fixes
* fix
* debug
* tweak
* fix
* fix timeout
* fix comment
* fixes
* x
* better test
* tests
* Automated change: Fix sanity tests
* missed file
* fix
* x
* fix
* fix
* fix
* fix
* Automated change: Fix sanity tests
* fix
* merge
* Automated change: Fix sanity tests
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
23da14c1a7
Revert "Revert "Revert "Implement transparent retries ( #28548 )" ( #28888 )" ( #28890 )" ( #28909 )
...
This reverts commit 9f90e6d906
3 years ago
9f90e6d906
Revert "Revert "Implement transparent retries ( #28548 )" ( #28888 )" ( #28890 )
...
* Revert "Revert "Implement transparent retries (#28548 )" (#28888 )"
This reverts commit 35708ff6b4
3 years ago
35708ff6b4
Revert "Implement transparent retries ( #28548 )" ( #28888 )
...
This reverts commit 7f8f3dc001
3 years ago
7f8f3dc001
Implement transparent retries ( #28548 )
...
* transport: add error attributes indicating stream network state
* add missing case
* transparent retries
* don't use backoff timer for transparent retries
* fix build
* add retry_transparent_not_sent_on_wire test
* add retry_transparent_goaway test
* remove special case to short-circuit retry code if no retry policy
* clang-format
* buildifier
* simplify logic a bit
* get StreamNetworkState from metadata instead of error
* fix api_fuzzer to always start recv_trailing_metadata_ready
* clang-format
* fix test flakiness in proxy fixture
* add test with MAX_CONCURRENT_STREAMS
* don't transparently retry if we're already committed
* fix test to not reuse byte buffers
* clang-format
* disable retries for streams_not_seen_test
3 years ago
b458db9246
Eliminate gRPC insecure build ( #25586 )
...
* force submit
* fix test error
* remove is_client from local tsi and its callsites
* fix too_many_pings_test
* add missing dep
3 years ago
9b42785db2
Removing Te metadata key-value pairs sent through initial or trailing metadata ( #28774 )
...
* Removing invalid Te metadata sent through initial or final metadata
* tidying up code to unlaterally remove Te metadata
3 years ago
9317838084
updating ChooseLbPolicy to revert to pick_first lb_policy if an unsupported lb_policy is passed through channel_args ( #28651 )
...
* updating ChooseLbPolicy to revert to pick_first lb_policy if an unsupported lb_policy is passed through channel_args
* addressing review comments
* updating comment
* adding check to ensure policy_name is not nullptr
* initializing bool variable to avoid ubsan errors
3 years ago
ff0ecd2ff4
sockaddr resolver: skip empty addresses in target URI ( #28695 )
...
* Adding constraints to the number of parsed targets in a specified URI to prevent OOMs
* removing hard limit on number of addresses in target uri
3 years ago
d02a68016e
Channel stack builder name needs to be static ( #28711 )
...
* Channel stack builder name needs to be static for stream refcount tracing
* Add test
3 years ago
74a2cb6e2b
Revert "Revert "Reland slice changes ( #28601 )" ( #28615 )" ( #28624 )
...
* Revert "Revert "Reland slice changes (#28601 )" (#28615 )"
This reverts commit 939bbfc336
3 years ago
939bbfc336
Revert "Reland slice changes ( #28601 )" ( #28615 )
...
This reverts commit b33e0d40af
3 years ago
b33e0d40af
Reland slice changes ( #28601 )
...
* Revert "Revert "Eliminate slice interning (#28363 )" (#28598 )"
This reverts commit 03bf699600
3 years ago
03bf699600
Revert "Eliminate slice interning ( #28363 )" ( #28598 )
...
This reverts commit 6703186b7a
3 years ago
6703186b7a
Eliminate slice interning ( #28363 )
...
Eliminate slice interning, and structures in slices to support it.
Reduces grpc_slice_refcount from 40 bytes (+ a required 8 bytes elsewhere) to 16 bytes.
Removes a pointer dereference for every slice ref/unref.
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
4b881c5947
Remove grpc_mdelem ( #28267 )
...
* Automated change: Fix sanity tests
* content-type
* Automated change: Fix sanity tests
* clang-format
* fix
* Move colon prefixed metadata
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* try to fix windows failure
* try and scale sizes better
* ambiguity fix?
* wip metadatavalueasslice
* Fix status code for resource exhaustion
* Revert "Revert "Move a bunch of slice typed metadata to new system (#28107 )" (#28208 )"
This reverts commit 7717587063
3 years ago
55b365f287
Move resolver code around a little ( #27846 )
...
* move service config
* service config should not depend on grpc_base
* move resolver, deal with fallout
* Automated change: Fix sanity tests
* fix
* fix
* Automated change: Fix sanity tests
* review feedback
* review feedback
* fix
* Automated change: Fix sanity tests
* fix
* Automated change: Fix sanity tests
* fix vis
* fix
* fixes
* Automated change: Fix sanity tests
* visibility
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
7e9e503bd9
Revert "modify call.cc to remove grpc-timeout metadata on the server side ( #28419 )" ( #28455 )
...
This reverts commit 038ea2196d
3 years ago
038ea2196d
modify call.cc to remove grpc-timeout metadata on the server side ( #28419 )
...
* modify api fuzzer code to disallow specification of grpc-timout metadata on the server side
* reverting changes to api_fuzzer and changing call.cc to drop grpc-timeout metadata on the server side
* tidying up code
3 years ago
e8c790df71
Dont hpack very long paths ( #28436 )
3 years ago
0dda706907
fixing heap use after free bug due to mismanaged ref counting in passthru endpoint ( #28417 )
...
* fixing heap use after free bug due to mismanaged ref counting in passthru endpoint
* fix sanity checks
3 years ago
f6d392b623
Fix timeout bug in api fuzzer ( #28385 )
...
* Fix timeout bug in api fuzzer
* add example
3 years ago
99e833d751
Better handle invalid content-type ( #28377 )
3 years ago
fb54d04ae2
Fix handling of very long user-agent strings ( #28376 )
...
* fix encoding of very long user-agent strings
* fuzzer found example
3 years ago
1a8d2b6760
API to cancel grpc_resolve_address ( #27883 )
...
Add an API to cancel grpc_resolve_address
3 years ago
264c321d83
Move compression related metadata to the new system ( #28223 )
...
* Eliminate most of grpc_message metadata handling
* Eliminate most of host metadata handling
* Remove more callouts without fixing code
* fiiixes
* typo
* Automated change: Fix sanity tests
* try-shrink
* Automated change: Fix sanity tests
* size tweaks
* less tricks
* deunique
* commonize
* commonize
* Automated change: Fix sanity tests
* size tuning, fixes
* Automated change: Fix sanity tests
* fix
* size tuning, fixes
* remove constexpr
* fix
* reuse code
* fix
* tweak code
* more tweaks
* tell no lies
* fixes
* fixes
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* fix?
* fix binder
* fix
* fix
* fixes
* Automated change: Fix sanity tests
* fix
* initial refactoring
* optimize status encoding
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* content-type
* Automated change: Fix sanity tests
* clang-format
* fix
* Move colon prefixed metadata
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* try to fix windows failure
* try and scale sizes better
* ambiguity fix?
* wip metadatavalueasslice
* Fix status code for resource exhaustion
* Revert "Revert "Move a bunch of slice typed metadata to new system (#28107 )" (#28208 )"
This reverts commit 7717587063
3 years ago
0deb64d1f6
Move content-type, colon prefixed metadata to new system ( #28204 )
...
* Eliminate most of grpc_message metadata handling
* Eliminate most of host metadata handling
* Remove more callouts without fixing code
* fiiixes
* typo
* Automated change: Fix sanity tests
* try-shrink
* Automated change: Fix sanity tests
* size tweaks
* less tricks
* deunique
* commonize
* commonize
* Automated change: Fix sanity tests
* size tuning, fixes
* Automated change: Fix sanity tests
* fix
* size tuning, fixes
* remove constexpr
* fix
* reuse code
* fix
* tweak code
* more tweaks
* tell no lies
* fixes
* fixes
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* fix?
* fix binder
* fix
* fix
* fixes
* Automated change: Fix sanity tests
* fix
* initial refactoring
* optimize status encoding
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* content-type
* Automated change: Fix sanity tests
* clang-format
* fix
* Move colon prefixed metadata
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* try to fix windows failure
* try and scale sizes better
* ambiguity fix?
* wip metadatavalueasslice
* Fix status code for resource exhaustion
* Revert "Revert "Move a bunch of slice typed metadata to new system (#28107 )" (#28208 )"
This reverts commit 7717587063
3 years ago
790365f261
xds: fix memory leak ( #28327 )
3 years ago
Craig Tiller
AJ Heller
Vignesh Babu
Esun Kim
Yousuk Seung
Mark D. Roth
yihuaz
Jan Tattermusch
apolcyn