Sent data handler, or socket/tls write error handling about libstrophe

[pasis]

Libstrophe is low-level library and this applies some restrictions. We can't simply resend items, because they may be valid only within current session. Resending is responsibility of user (user of the library, program in most cases).
Second, any change must not break existent programs.

You're right, would be nice to have some "reliable" interface or access to unsent items. You can try to play around connection_handler. User notified with event XMPP_CONN_DISCONNECT from conn_disconnect() on errors. And at this point conn contains unsent items. Just a quick thought: you can make interface xmpp_conn_unsent_nr()/next() and user can go through the messages and save ones need to be resent. But user doesn't work with raw strings, so I think next() should return xmpp_stanza_t. Libstrophe stores complete stanzas in raw strings in the send_queue what makes this approach possible.

Also, as you said, send_queue should be cleared on a connection error. Since connection_handler may want to access the queue and call xmpp_connect_client() we can't clear queue neither before nor after handler's call in conn_disconnect(). But we can clear it at the top of connect-like functions with some static function conn_send_queue_purge().

What about handler for successfully sent items: user can mix xmpp_send() and xmpp_send_raw().

[stanson-ch]

User notified with event XMPP_CONN_DISCONNECT from conn_disconnect() on errors. And at this point conn contains unsent items. Just a quick thought: you can make interface xmpp_conn_unsent_nr()/next() and user can go through the messages and save ones need to be resent.

Good point, looks like solution. Will try.

Also, as you said, send_queue should be cleared on a connection error. Since connection_handler may want to access the queue and call xmpp_connect_client() we can't clear queue neither before nor after handler's call in conn_disconnect(). But we can clear it at the top of connect-like functions with some static function conn_send_queue_purge()

Purging queue in xmpp_connect_client is a good idea. This will allow to make "short cycle" reconnects possible, without releasing connection, creating new connection and setting same credentials again. Will try to make a patch.

[pasis]

Fix for the send_queue is already in master and reconnect works. Access to unsent data is not in priority for the next release (0.9), but may be added in some future release.