search

matrix-org / matrix-spec

The Matrix protocol specification
Apache License 2.0
193 stars 96 forks source link

delete endpoint for pushers #233

Open mujx opened 7 years ago

mujx commented 7 years ago

The description of the kind parameter from the current version.

The kind of pusher to configure. "http" makes a pusher that sends HTTP pokes. null deletes the pusher.

I would suggest creating a new endpoint like DELETE /pusher to delete a pusher and be more explicit about the intend instead of hiding it inside an arbitrary parameter.

turt2live commented 5 years ago

Not to mention it makes it nearly impossible to troubleshoot as a client dev whether your pusher is being deleted or added successfully by the server. 200 OK just means it did something, but it is hard to tell what at a glance.

richvdh commented 1 year ago

Another reason for this is that there's basically nowhere else in the protocol that we allow null (cf #472)