marc0shi, what's the purpose of the `max` attribute of the 0198's <enable/>? I cannot see anything more than client's/server's "preferred maximum resumption time" in the XEP on what to do about it? Am I missing something?
marc0sShould I infer the server should not allow a resume once that `max` time passes? May the client do the same and do not even try to resume the expired stream?
Zashmarc0s: Presumably so the client can say "please keep my session all eternity" and the server can reply "haha no, you get 5 minutes"
flowZash, marc0s: I think it's more like the client requesting the maximum allowed maximum resumption time
flowZash, marc0s: I think it's about enabling the client to request the maximum allowed maximum resumption time without a-priori knowledge what that time may be
flowakin to 'max' in max pubsub node items
marc0sZash, I can see that. And should the client do some local check to avoid asking for an already-known-to-be-expired or better wait for the server to deny it? Would it make the client a good citizen? :) Maybe it's just out of spec...
Zashmarc0s: Personally I would try to resume regardless of that value, as I would try to at least save the 'h' value for as long as possible so you can know what, if anything, was lost in the last disconnect.
marc0sYes, I'd do that too
ZashNo idea how things work in the wild tho.
marc0sI was just seeing how XMPPFramework behaves, and it just refuses to resume if max already passed
flowfwiw, that's also what smack does, mostly because nobody changed that ever since the initial impl
flowfwiw, I think that's also what smack does, mostly because nobody changed that ever since the initial impl
ZashWithout looking at any code, have a vauge idea that mod_smacks for prosody has integration with the mobile push notification stuff, such that sessions are kept forever if they're set up to send push notifications.
ZashSo then refusing to resume would be silly
ZashAnd also sending a 'max' of 10 mintues would be silly
marc0sZash, indeed, having mod_cloud_notify makes hibernation permantent given you have enabled push notifications
flowZash, "kept forever" smells like a memory leak… ;)
Zashflow: push notifications and smacks? you have no idea.... nor do I, I just stay away from that code
marc0sthanks for your comments :)
ZashAn unpublished 198 implementation I wrote saved resumption id + 'h' value to persistent storage so you'd be able to at least get a failure + 'h' even after a crash or whatever.