-
Notifications
You must be signed in to change notification settings - Fork 3
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
Guidelines for subcommands and/or numerics #6
Comments
Just for some context from the client dev perspective:
|
A standard specification for how errors are defined and returned should exist. Custom numerics are limited in number, hard to track, and non-intuitive. The ability for command specifications to define reply codes by name (ACCESS_DENIED, CMD_INVALID, etc...) would allow for clients to better handle replies. Clients supporting the extension / capability / command would be required to handle each supported reply message. The reply could be returned as a sub-command. Expanding on and generalizing @Zarthus's idea, a standard server reply in the format For instances where the same command may be sent multiple times, draft/labeled-replies could be implemented into the command, providing for identification of the specific command instance sent to the server (
Likewise, how these replies are defined should be addressed. Are reply types and/or codes standard or custom? Are types standard with custom codes? If standardized, what is the process for defining a new reply and introducing it into the spec. |
Made ircv3/ircv3-specifications#357 in an attempt to start resolving this on the making new error responses side at least. |
IRCv3 is rather vague on how create new subcommand responses and add new numerics, guidelines and/or style guides on how to handle this would help ending up with inconsistencies in the future
Small sample:
A command /TEST returns the subcommand response.
TEST ERR SOME_ERROR
Guidelines could exist for "the error should always be ERR, not ERROR", "SOME_ERROR" has to be in all caps, and "on success just return the result."
In a similar vein, we don't know whether we're supposed to pull numerics out of our arse either.
The text was updated successfully, but these errors were encountered: