Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Server-side multi-protocol support #2979

Open
david-perez opened this issue Sep 12, 2023 · 2 comments
Open

Server-side multi-protocol support #2979

david-perez opened this issue Sep 12, 2023 · 2 comments
Labels
enhancement New feature or request high-priority High priority issue server Rust server SDK

Comments

@david-perez
Copy link
Contributor

Smithy service shapes can be annotated with multiple protocol traits.

$version: "2"

namespace smithy.example

use aws.protocols#restJson1
use aws.protocols#awsJson1_1

@restJson1
@awsJson1_1
service MyService {
    version: "2023-09-12"
}

In theory this indicates that clients invoking operations of MyService can use any of the two restJson1, @awsJson1_1 protocols, and MyService should accept incoming requests in any of the two protocols.

Currently server SDKs are generated such that they only support the first protocol the service shape is annotated with; the rest of the protocols are ignored.

Multi-protcol support is important for services that want to begin supporting a better protocol and migrate their existing clients to it. Without a period during which the service understands both protocols, the migration becomes challenging or even impossible.


Implementation-wise, one possiblity is that when receiving a request, the server SDK iterates over a priority-ordered list of supported protocols and attempt to "claim" it: if claiming succeeds, the request is handed over to that protocol's router. If routing or any other step thereafter fails, the request is rejected: we do not attempt to process the request with the next protocol.

When tackling this issue I'd like the algorithm to be detailedly documented, and I'd like as much of it to be contributed to the Smithy specification so that other Smithy-based server SDK generators abide by the same rules. In particular, we should answer these questions:

  1. In what order do servers claim a request for a protocol?
    • Perhaps this should not be publicly documented and left implementation-defined, so that users don't attempt to rely on it in some way. The order should be deterministic though.
  2. How does each of our supported protocols attempt to claim a request? Ideally, this check should be, simple, inexpensive, and not modify the request or attempt to deserialize it.
    • For example, for the awsJson1_1 protocol, checking first the HTTP version against the http and eventStreamHttp properties and then checking for Content-Type equalling application/x-amz-json-1.1 should suffice to unambiguously detect that the request is indeed meant to be processed by said protocol (one could additionally and inexpensively check for the URI being /, method being POST, and the presence of X-Amz-Target, so it's important that a decision is made and the concrete steps be documented, ideally in the spec). For other protocols, like restJson1, the checks to perform are less apparent.
  3. Could Smithy help the user by failing at build time in not defining operations where a request could be claimed by multiple protocols i.e. "ambiguous requests"?
  4. What does a server respond with if the request cannot be claimed by any annotated protocol? Most likely with a 404 and an UnknownOperationException, but serializing said exception is kinda unsolvable when you haven't agreed on a protocol.
  5. What happens with operations that are only available in specific protocols, because they rely on features that are not available in all the annotated protocols (e.g. event streams)?
@david-perez
Copy link
Contributor Author

Asking Smithy for guidance: smithy-lang/smithy#2123

@drganjoo drganjoo added the high-priority High priority issue label Apr 10, 2024
@david-perez
Copy link
Contributor Author

As we start adding support for new protocols (#3573), this task gains importance: existing services will want to make use of more modern protocols.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request high-priority High priority issue server Rust server SDK
Projects
None yet
Development

No branches or pull requests

2 participants