search

mozilla-services / syncstorage-rs

Sync Storage server in Rust
Mozilla Public License 2.0
979 stars 49 forks source link

Add dynamic routing headers to the Spanner client #1490

Closed data-sync-user closed 1 year ago

data-sync-user commented 1 year ago

Per a recent incident we’ve determined our client does not support “dynamic routing”.

This feature appears to consist of an additional gRPC metadata (header) added to certain requests. This header is defined in AIP-4222 Routing headers.

Spanner’s current protobuf definition includes some usages of the implicit google.api.http header. Spanner API calls with such header definitions should result in added x-goog-request-params gRPC metadata headers added to their requests.

┆Issue is synchronized with this Jira Task

data-sync-user commented 1 year ago

➤ Philip Jenvey commented:

Some minor notes: https://docs.google.com/document/d/1tCnnYDruFONsRGs4ykQfPYKxbuicU7hZMVdkzMX71Fk/edit ( https://docs.google.com/document/d/1tCnnYDruFONsRGs4ykQfPYKxbuicU7hZMVdkzMX71Fk/edit|smart-link )