Craig Tiller
cadce47ebf
Convert client_auth_filter to promises ( #28767 )
...
* Convert client_auth_filter to promises
* outline
* x
* x
* fixes
* remove-spam
* Async sequences over iterators
* iterate
* better semantics
* fix
* clang-tidy
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fixes
* fixes
* Automated change: Fix sanity tests
* progress
* progress
* progress
* progress
* progress
* credentials_test passes
* Automated change: Fix sanity tests
* Make Activity more of an interface
Move mutex and wakeup logic into a new class between PromiseActivity<>
and Activity (so that the sharing can persist), but make Activity closer
to a pure interface, so that whilst we migrate code we can implement
better fakes without forcing allocation.
* fixes
* Automated change: Fix sanity tests
* fixes
* test the awful failure
* x
* fix typo
* fix race
* 4.9
* x
* review feedback
* review feedback
* Automated change: Fix sanity tests
* review feedback
* replace AuthMetadataContext interface with GetRequestMetadataArgs struct
* Automated change: Fix sanity tests
* fix merge
* Fixup
* fix
* fix
* fix
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Mark D. Roth <roth@google.com>
3 years ago
Mark D. Roth
f48c4f1f4b
retry: fix memory leak due to not freeing cached send ops upon cancellation ( #28945 )
...
* retry: fix memory leak due to not freeing cached send ops upon cancellation
* clang-format
* attempt to fix proxy tests on windows
3 years ago
Craig Tiller
dd76a04b8c
Move resolver to core configuration ( #28881 )
...
* Move resolver to core configuration
* Automated change: Fix sanity tests
* Automated change: Fix sanity tests
* fix
* fix
* fix
* fix
* resolver: clean up and modernize registry
* fix race
* update visibility
* fix internal error
* review feedback
* resolve backref issues
* windows
* x
* fix sockaddrs on windows?
* fix sockaddrs on windows?
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
Co-authored-by: Mark D. Roth <roth@google.com>
3 years ago
Craig Tiller
ff87ca02b2
Revive test/core/memory_usage ( #28931 )
...
* Revert "Deprecate gpr_get/set_allocation_functions"
This reverts commit 81df105ac8
.
* memory stats
* Hack up to get per-call numbers
* expunge memory counters
* buildgen
* Automated change: Fix sanity tests
* sanity
* fix
* Automated change: Fix sanity tests
* simplify
* better
* Automated change: Fix sanity tests
* Disable windows memory_usage
* disable mac
* disable mac
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Mark D. Roth
be70a8676c
Revert "Third attempt: Implement transparent retries ( #28925 )" ( #28942 )
...
This reverts commit 9e7453409a
.
3 years ago
Yash Tibrewal
3f19e0cc02
Revert "Revert "CompositeChannelCredentials: Comparator implementation ( #28902 )" ( #28919 )" ( #28918 )
...
* CompositeChannelCredentials: Comparator implementation retry
* Fix test
3 years ago
Mark D. Roth
9e7453409a
Third attempt: Implement transparent retries ( #28925 )
...
* Revert "Revert "Revert "Revert "Implement transparent retries (#28548 )" (#28888 )" (#28890 )" (#28909 )"
This reverts commit 23da14c1a7
.
* revert api_fuzzer change
3 years ago
AJ Heller
85189b24bb
Reintroduce: Avoid fully qualifying namespaces (and add check) ( #28917 )
...
Based on a handful of https://abseil.io/tips , it's generally advised to
only fully-qualify namespaces when in a `using` statement, or when it's
otherwise required for compilation. In all other cases, the general
recommendation is to not fully-qualify.
This change fixes most `grpc.*` namespace uses. There are potential
challenges in trying to make blanket changes to non-gRPC namespace uses,
such as `::testing`, since there is also a `grpc::testing` namespace.
3 years ago
Yash Tibrewal
bbfcb7ca35
Revert CompositeChannelCreds merge fiasco ( #28930 )
3 years ago
Yash Tibrewal
778ad437b1
HttpRequestSSLCredentials: Comparator implementation ( #28900 )
...
* HttpRequestSSLCredentials: Comparator implementation
* Unused argument
* Singleton HttpRequestSSLCredentials
3 years ago
Yash Tibrewal
a72c8ebb7d
CredentialsTest: Fix TestGoogleDefaultCredsExternalAccountCredentials ( #28923 )
3 years ago
Mark D. Roth
25aa80966a
retry: fix memory leak if call is unreffed without starting recv_trailing_metadata ( #28913 )
...
* retry: fix memory leak if call is unreffed without starting recv_trailing_metadata
* fix test flake
3 years ago
Yash Tibrewal
e6a54ef372
Revert "CompositeChannelCredentials: Comparator implementation ( #28902 )" ( #28919 )
...
This reverts commit 2fdb5f90c3
.
3 years ago
AJ Heller
e72a5fe5dd
Revert "Avoid fully qualifying namespaces (and add check) ( #28901 )" ( #28916 )
...
This reverts commit fc7314c701
.
3 years ago
AJ Heller
fc7314c701
Avoid fully qualifying namespaces (and add check) ( #28901 )
...
Based on a handful of https://abseil.io/tips , it's generally advised to
only fully-qualify namespaces when in a `using` statement, or when it's
otherwise required for compilation. In all other cases, the general
recommendation is to not fully-qualify.
This change fixes most `grpc.*` namespace uses. There are potential
challenges in trying to make blanket changes to non-gRPC namespace uses,
such as `::testing`, since there is also a `grpc::testing` namespace.
3 years ago
Yash Tibrewal
2fdb5f90c3
CompositeChannelCredentials: Comparator implementation ( #28902 )
3 years ago
Mark D. Roth
23da14c1a7
Revert "Revert "Revert "Implement transparent retries ( #28548 )" ( #28888 )" ( #28890 )" ( #28909 )
...
This reverts commit 9f90e6d906
.
3 years ago
Yash Tibrewal
e178a44931
XdsCredentials: Comparator implementation ( #28903 )
3 years ago
Craig Tiller
0f424ae5cc
Move client-idle-filter to promises ( #28838 )
...
* Promise based sleep
* Embrace absl::Status
* Automated change: Fix sanity tests
* Add another test, fix bug
* fix
* fix
* Move client-idle-filter to promises
* better call counting
* x
* x
* x
* review feedback
* progress
* Automated change: Fix sanity tests
* progress
* Automated change: Fix sanity tests
* fix
* channel stack refcount
* remove op from lock, refcount
* compiles
* Automated change: Fix sanity tests
* nicer api
* fix
* Automated change: Fix sanity tests
* fix
* speculative fix for windows
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Yash Tibrewal
95c12d88c3
InsecureCredentials : Comparator implementation ( #28899 )
...
* InsecureCredentials: Update cmp_impl
* Fix tests
* Unused argument
3 years ago
Yash Tibrewal
c65a55383a
Gtestify credentials_test ( #28897 )
3 years ago
Mark D. Roth
9f90e6d906
Revert "Revert "Implement transparent retries ( #28548 )" ( #28888 )" ( #28890 )
...
* Revert "Revert "Implement transparent retries (#28548 )" (#28888 )"
This reverts commit 35708ff6b4
.
* disable retries for ALTS handshaker channel
3 years ago
AJ Heller
5ef1585070
A more granular EventEngine conformance test suite ( #28889 )
...
This allows implementers to select which subset of the conformance test
suite they wish to exercise with their implementation. This was a
request from the fuchsia team, and may be useful for partial
implementations that are composed into a complete EventEngine solution.
3 years ago
Yash Tibrewal
5f67cd07f4
Explicit method for comparing channel credentials ( #28844 )
...
* Use an explicit cmp method on grpc_channel_credentials
* Add testing
* Add TODOs to improve cmp methods
* Automated change: Fix sanity tests
* clang format
* Reviewer comments
* clang-format
* Add cmp method for grpc_call_credentials
* s/overriden/overridden
Co-authored-by: yashykt <yashykt@users.noreply.github.com>
3 years ago
Mark D. Roth
35708ff6b4
Revert "Implement transparent retries ( #28548 )" ( #28888 )
...
This reverts commit 7f8f3dc001
.
3 years ago
AJ Heller
84ed11459d
Delete custom iomgr ( #28816 )
...
* Squashed version of gnossen's gevent pR
* rm
* Automated change: Fix sanity tests
Co-authored-by: drfloob <drfloob@users.noreply.github.com>
3 years ago
Ashitha Santhosh
1fee3d72be
remove SDK term from gRPC authz ( #28843 )
3 years ago
Mark D. Roth
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
Ashitha Santhosh
55e4af1b14
Revert "Revert "Update to rbac policy struct and end2end authz test. ( #27074 )" ( #28552 )" ( #28620 )
...
This reverts commit 8ca42ec6f8
.
3 years ago
Craig Tiller
df943da2c4
Promise based sleeps ( #28722 )
...
* Promise based sleep
* Embrace absl::Status
* Automated change: Fix sanity tests
* Add another test, fix bug
* fix
* fix
* review feedback
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Yash Tibrewal
20aec3b2c1
Set trailing_metadata_available for recv_initial_metadata ops when generating a fake status ( #28827 )
...
* Set trailing_metadata_available for recv_initial_metadata ops when generating a fake status
* Remove log
* Fix
* Revert "Convert filter to a promise (#28815 )"
This reverts commit 361809aabb
.
* Add testing
3 years ago
Craig Tiller
9cb0747ab0
Improve promise-based-filter APIs ( #28839 )
...
* Improve promise-based-filter APIs
* review feedback
* comment
* fix
3 years ago
yihuaz
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
Yousuk Seung
172120f6b4
Move XdsChannelCreds to CoreConfiguration ( #28746 )
...
* Move XdsChannelCreds to CoreConfiguration
* move xDS channel creds files to src/core/lib/security/credentials/xds
* Change back to returning a RefCountedPtr.
* make remove "xds_" from xds_channel_* files.
* Renamed to address comments.
* clang fix
* Fix another clang error
3 years ago
Craig Tiller
4169f24dcc
Revert "Revert "Transport channel arg ( #28802 )" ( #28818 )" ( #28820 )
...
This reverts commit 2532cf5321
.
3 years ago
Craig Tiller
361809aabb
Convert filter to a promise ( #28815 )
...
* Convert filter to a promise
* copy/paste fix
* fix
3 years ago
Craig Tiller
2532cf5321
Revert "Transport channel arg ( #28802 )" ( #28818 )
...
This reverts commit 99e339136d
.
3 years ago
Craig Tiller
f55c7ebeb4
Non-encodable tweaking ( #28733 )
...
* log non-encodables
* add a place to capture errors into trailing metadata without risking encoding it
* bs
3 years ago
Craig Tiller
99e339136d
Transport channel arg ( #28802 )
...
Instead of passing transport optionally as an argument adjacent to channel args, pass it as a channel arg directly.
Doing so does not affect semantics, but does allow a cleaner API for channel creation which will become increasingly important as we move towards promises.
Co-authored-by: ctiller <ctiller@users.noreply.github.com>
3 years ago
Esun Kim
84101427d0
Fix CFStreamEndpointTests ( #28812 )
3 years ago
Mark D. Roth
d5fc37f706
rls: add routeLookupChannelServiceConfig field to LB policy config ( #28731 )
...
* rls: add routeLookupChannelServiceConfig field to LB policy config
* fix error refcount bug
3 years ago
Esun Kim
5a4ba15346
Fix build & test errors when `GRPC_ERROR_IS_ABSEIL_STATUS` enabled. ( #28784 )
...
* Fix xds_bootstrap_test
* Fix sanity test
* Fix ev_apple
* Fix debug_location
3 years ago
apolcyn
64082940a5
Fix google c2p resolver shutdown during metadata server queries ( #28519 )
...
* Fix c2p resolver shutdown during metadata server queries
* handle lame channels in XDS client
3 years ago
Yash Tibrewal
9ffd1a7b0a
Fix for a racy WorkSerializer shutdown ( #28769 )
...
* Fix for a racy WorkSerializer shutdown
* Reviewer comments
* Additional test
* Fix test compilation on cmake
3 years ago
krestofur
b8d3a09092
Update CRL test credentials ( #28794 )
...
* remove old files
* update credentials
* fix README
3 years ago
Yash Tibrewal
a8ba47ac55
InsecureCredentials: singleton object ( #28777 )
...
* InsecureCredentials: Allow special case comparison
* Update security connector
* Use a singleton object instead
3 years ago
Esun Kim
342cb4457c
Added a temporary trap to prevent Abseil-Status breakages. ( #28766 )
...
* Added a new trap to prevent build errors with use_abseil_status enabled
* Fix build errors.
3 years ago
Vignesh Babu
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
Esun Kim
92738290ab
Upgrade base-builder to the latest for blaze build ( #28768 )
...
* Upgrade base-builder to the latest for blaze build
* Fix warnings.
* Added -Wno-deprecated-copy
3 years ago
Vignesh Babu
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