openid / sharedsignals

OpenID Shared Signals Working Group Repository
45 stars 11 forks source link

7.1.2.2 Update Stream Status (inconsistent use of HTTP POST for modify vs. create) #113

Closed independentid closed 2 months ago

independentid commented 10 months ago

In stream management, HTTP POST is used to create a new stream and PUT/PATCH is used to update.

Yet in stream status, POST is used to update.

Recommend for consistency supporting PUT to update the stream status. The use of POST could be deprecated to preserve backward compatibility.

This is not a big deal. But it is an inconsistency with matching CRUD operations to HTTP POST, GET, PUT, DELETE.

tulshi commented 2 months ago

Since this is going to be a breaking change, I'd like to say we won't be fixing this one, and closing this issue. If anyone feels we should still do this, please reopen this issue.