-----BEGIN PGP SIGNED MESSAGE----- Hash: SHA512 On 04.12.2014 18:07, Kim Alvefur wrote: > On 2014-12-04 15:11, Kamil Kisiel wrote: >> On Thu, Dec 4, 2014 at 6:00 AM, Dave Cridland <[email protected] >> <mailto:[email protected]>> wrote: That's how it was specified in >> version 0.2, it seems it was changed to <fin/> in 0.3 > > I remember someone argued very persistently that this change was > needed because their code had timeouts for iq requests that could > trigger if there is rate limiting or a slow connection before all > the messages and the iq-reply was received. > > Or something. Personally I prefer having the iq-result sent last, > it makes client code (Verse in my case) simpler.
Exactly my experience. I recently started working on xep313 in Smack, and if the iq response would signal the end of the messages the code would become less complex. I also don't see how using a message compared to an iq response yields any benefit when working on the xep313 client code. Only if your iq response timeouts are hard-coded maybe, but this appears to be a bad idea anyway. It seems like a good idea for every xep313 client API to provide an optional 'timeout' parameter, which defaults to the connections default reply timeout, in order to give the user at least a chance to work around problems caused by e.g. rate limiting or slow connections. And in the end it doesn't make a difference if you use that timeout waiting for the message with the <fin/> extension or the iq response. Ideally xep313 implementations reset the timeout waiting for the final iq response every time an archived message is received. As a side note: xep13 uses the same approach (see ยง 2.6, example 11 and 12). - - Florian -----BEGIN PGP SIGNATURE----- Version: GnuPG v2 Comment: Using GnuPG with Thunderbird - http://www.enigmail.net/ iQF8BAEBCgBmBQJUtqOSXxSAAAAAAC4AKGlzc3Vlci1mcHJAbm90YXRpb25zLm9w ZW5wZ3AuZmlmdGhob3JzZW1hbi5uZXQxMzU3QjAxODY1QjI1MDNDMTg0NTNEMjA4 Q0FDMkE5Njc4NTQ4RTM1AAoJEIysKpZ4VI414SUIAIrUFRLRTcR8gaRzsMx51JX5 uaPrB8q7zaJccHsQU/j8B3zFfy4hNnU1UzYOwVcGoLY4FJ8ZeUdBhRzPmJ27MKfv 6q7tliat78RsCqDsgoWTSa777H0P9Qljv4dhWssjAJJlDGJ3x4z0AWKZ9oEbh+8U NKmf9b+OtT2vhWYksDFXm+NjI2APXfAwzRhFVIYmxo9XRXALa+8wVv0gyGGQmYCW zDRwZaigCRZ6n4OZyNXoPBTrHYPJKYgEeeUCNvQs+CEaOj8kOe9UWfTdEuaG8Wfa ZmjfR0mAaqYR8WTopZ6p7xmLqLJOzMz2QjEFz4iIIS438Ew6ZAhOY7AuuvY3pmQ= =90ID -----END PGP SIGNATURE-----
