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

Stream creation lanaguage imply that transmiter must suport the poll delivery method if delivery method is not specified #149

Open
ysarig1 opened this issue Apr 2, 2024 · 1 comment
Labels
bug Something isn't working spec:SSF vFuture

Comments

@ysarig1
Copy link
Collaborator

ysarig1 commented Apr 2, 2024

Section 7.1.1.1 (stream creation) have the following language: "If the request does not contain the delivery property, then the Transmitter MUST assume that the method is "urn:ietf:rfc:8936" (poll). The Transmitter MUST include a delivery property in the response with this method property and an endpoint_url property."

The language seem to imply that the transmitter must support the poll method, which shouldn't be the case as supporting the poll method is optional.

The language should be changed that a 400 response should be returned if the poll method is not supported when delivery is not specified in the request.

@tulshi tulshi added bug Something isn't working spec:SSF labels Apr 20, 2024
@FragLegs FragLegs added the vID-03 Address this issue before the v3 cutoff (June 15, 2024) label May 9, 2024
@tulshi
Copy link
Contributor

tulshi commented May 23, 2024

This is a backward-incompatible change, so we should not include this in v3.

@tulshi tulshi removed the vID-03 Address this issue before the v3 cutoff (June 15, 2024) label May 23, 2024
@timcappalli timcappalli added v1 Issues required to be fixed before v1 vFuture and removed v1 Issues required to be fixed before v1 labels Oct 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working spec:SSF vFuture
Projects
None yet
Development

No branches or pull requests

4 participants