-
Notifications
You must be signed in to change notification settings - Fork 428
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
use exml library for parsing XML #40
Comments
The main rationale behind switching to The current state of XML handling in ejabberd is that it doesn't store the NS (namespace) of every element, but in general assumes that the NS is What it means in practice is that we deal just right with the following example of XMPP stream:
but wouldn't be able to cope with equally legal in terms of the standard example:
Firstly, some XMPP client might prefer the latter formulation and current ejabberd just won't understand it. Secondly, it might prove to be a source of network usage optimization to declare often used namespaces only once and then use just the short prefixes for specific elements. |
Further work is done in #183 |
As reported on the hive by Raffaele ``` I found a bug or undesired behavior with the Inbox module. So when we fetch the Inbox, we can order it by date right? Every Inbox response has this "delay" element with a timestamp. However, it looks like when we perform any of the follwing actions, it resets that timestamp to the current time: - Archiving/unarchiving - Muting/unmuting - Marking as read/unread So that means that if, for example, I mute a conversation, then it gets bumped to the top of my inbox which is a bit strange. ``` The fix simply removes setting the timestamp, which I shouldn't have done to begin with, but for the query to be valid I need to do some simple comma-separator fixing.
No description provided.
The text was updated successfully, but these errors were encountered: