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

ANNOUNCE error codes #393

Open
vasilvv opened this issue Feb 19, 2024 · 4 comments
Open

ANNOUNCE error codes #393

vasilvv opened this issue Feb 19, 2024 · 4 comments
Labels
Announce Issues with Announce message and handling Parked Issue we may discuss later or close as OBE

Comments

@vasilvv
Copy link
Collaborator

vasilvv commented Feb 19, 2024

We currently don't have any specific error codes specified for announce. We should either define some, or merge the space with subscribe errors.

The ones I have in mind are "Internal error", "Announce not supported" (aka this is a delivery-only endpoint), "Permission denied".

@ianswett ianswett added the Announce Issues with Announce message and handling label Feb 19, 2024
@fluffy
Copy link
Contributor

fluffy commented Feb 21, 2024

+1

What does the sender do with an "internal Error" ?

@kixelated
Copy link
Collaborator

+1

Most of these errors are solely informative since we don't have a REASON string. At a minimum we need "unauthorized" which could be automatically handled by the publisher, but more often will just be presented to the user.

@vasilvv
Copy link
Collaborator Author

vasilvv commented Feb 21, 2024

What does the sender do with an "internal Error" ?

Probably the same thing one would do in HTTP when you receive a 500 error (which is typically tell the user their operation cannot be finished because the server is broken).

@ianswett
Copy link
Collaborator

ANNOUNCE_CANCEL now has error codes as well (#522), so ideally we should fix those at the same time.

@ianswett ianswett added the Parked Issue we may discuss later or close as OBE label Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Announce Issues with Announce message and handling Parked Issue we may discuss later or close as OBE
Projects
None yet
Development

No branches or pull requests

4 participants