search

braid-org / braid-spec

Working area for Braid extensions to HTTP
https://braid.org
233 stars 16 forks source link

Simplify `Subscribe: keep-alive` #122

Closed toomim closed 1 year ago

toomim commented 1 year ago

Following the result of issue https://github.com/braid-org/braid-spec/issues/80, this removes the keep-alive parameter from the Subscribe: header, since we haven't fully taken on the design of Subscription Parameters yet.

We will design these parameters later.

Note that I made the examples use the blank Subscribe: header, rather than Subscribe: true, because I don't want to mislead implementations into requiring true as a value, because that could prevent clients from adding new parameters in the future without risking servers ignoring their subscription requests rather than simply ignoring optional parameters.

toomim commented 1 year ago

This PR is small, and has received some support and no objections. Merging.