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

Protocol v3 (?) will add mandatory version parameter to REST: Opaque Request #90

Open
QuintinWillison opened this issue Oct 20, 2022 · 3 comments
Labels
enhancement New feature or request

Comments

@QuintinWillison
Copy link
Contributor

Most of our client libraries advertise compliance with this feature as specified by RSC19.

ably/specification#89: Protocol v2 adds RSC19f, replacing RSC19a, fundamentally changing the way that this API is used.

I think this warrants a new feature node. Perhaps as a child of the existing node:

REST: Opaque Request: with Version

Or as a new sibling of the existing node, introducing deprecation of the existing node (a minor bump to this repository) or replacing it entirely (a major bump to this repository):

REST: Opaque Request with Version

Work on this issue will involve making that decision, so that this new feature node is made available to downstream SDK repositories to appear in their compliance manifests.

@QuintinWillison QuintinWillison added the enhancement New feature or request label Oct 20, 2022
@sync-by-unito
Copy link

sync-by-unito bot commented Oct 20, 2022

➤ Automation for Jira commented:

The link to the corresponding Jira issue is https://ably.atlassian.net/browse/SDK-2847

@SimonWoolf
Copy link
Member

We might also want to do a wider redesign of rest#request (which is currently very weird) at the same time as this, given we have to break the api anyway for this. CF discussion at ably/specification#103

@QuintinWillison QuintinWillison changed the title Protocol v2 adds mandatory version parameter to REST: Opaque Request Protocol v3 (?) will add mandatory version parameter to REST: Opaque Request Dec 5, 2022
@QuintinWillison
Copy link
Contributor Author

I've modified the title of this issue to reflect the recent decision that this change will be separated from the no-connection-serial work (which is v2, making this a v3 change to the wire protocol). cc @SimonWoolf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

No branches or pull requests

2 participants