#1325 XEP-0313 (mam): Prosody is not returned item-not-found when an unknown item id is used in <after> or <before>
Reporter
Goffi
Owner
Zash
Created
Updated
Stars
★ (1)
Tags
Compliance
Milestone-0.12
Priority-Medium
Type-Defect
Status-Fixed
Goffi
on
Hello,
Prosody is not returned an item-not-found error when an unknown item is used in <after> or <before>.
--
Please provide a snippet of the part of the specification which we violate
if possible:
XEP-0313 states that "If the UID contained within an <after> or <before> element is not present in the archive, the server MUST return an item-not-found error in response to the query." (§4.1.4)
Please provide a link to the specification:
https://xmpp.org/extensions/xep-0313.html#query-paging
--
Thanks
Zash
on
This was a recent change to the XEP that I did not notice.
The current behavior was introduced with #624 and is embedded in the archive API.
Changes
tags Status-Accepted
Zash
on
Have fixes but I'm not sure how to handle the transition since the fix is not to mod_mam, but every storage driver that currently exists.
Hello, Prosody is not returned an item-not-found error when an unknown item is used in <after> or <before>. -- Please provide a snippet of the part of the specification which we violate if possible: XEP-0313 states that "If the UID contained within an <after> or <before> element is not present in the archive, the server MUST return an item-not-found error in response to the query." (§4.1.4) Please provide a link to the specification: https://xmpp.org/extensions/xep-0313.html#query-paging -- Thanks
This was a recent change to the XEP that I did not notice. The current behavior was introduced with #624 and is embedded in the archive API.
ChangesHave fixes but I'm not sure how to handle the transition since the fix is not to mod_mam, but every storage driver that currently exists.
ChangesFixed in https://hg.prosody.im/trunk/rev/7408b9473729 .. https://hg.prosody.im/trunk/rev/b56654f89cd5 for varying parts of this. 3rd party storage modules may need similar fixes.
Changes