-
Sam
I know I've seen this discussed before, but I never remember what to do: ad-hoc commands says `The requester MAY include the "action='execute'", although this is implied.` so what is the point? Do things have problems if you add it or leave it off?
-
jonas’
oh no
-
jonas’
I get flashbacks whenever someone brings up Ad-Hoc’s @action
-
Sam
Yah, it's irritating and I never remember what the decision was every time I look at it
-
jonas’
"fubar"
-
Sam
meh, just going to leave it off and hope nothing breaks.
-
Sam
Same with cancelation. It returns a result, but why? Is there anything else it can do other than be a success?
-
Sam
(or an IQ error I guess, but at the ad-hoc level I Just don't understand why all this optional stuff that as far as I can tell is meaningless)
-
flow
Sam, you mean xep50 § 2.4.3? What else should the command return if the cancellation was successful?
-
Sam
just an empty iq result
-
flow
besides it can also be an error response, e.g. if the session is unknown
-
flow
ahh yes, the payload in case of the result IQ could be considered redundant
-
Sam
Right, it could be an IQ error, I just don't see the oint of having a payload if it was successful. It tells me nothing as far as I can tell, so why bother unmarshaling it at all?
-
flow
I believe there are some (rare) cases in XMPP where this helps the client to drop some state while the request is processed (or to not keep the state in the first place), but I don't think this is one of them
-
flow
so it may be simply a bad design :)
-
Sam
cool, thanks for the sanity check