Do not deliver offline messages to clients with negative priority

Description

http://tools.ietf.org/html/rfc3921#page-86 reads: "the server MUST NOT deliver the stanza to an available resource with a negative priority"
According to the bug report Openfire does deliver the messages when a client logs in, even if tit has a negative priority.

RFC 6121 has the same constraint in 8.5.2.1.1. Message

Environment

any

Activity

Show:
Tim Durden
January 7, 2016, 10:36 AM
csh
June 7, 2014, 4:56 PM

Tested with 3.10 and it was still an issue.
XEP-0160 also forbids this behavior.

Simon White
March 5, 2013, 7:42 AM
Edited

Guus,

I made the change there simply because I wasn't 100% sure of the contract of canFloodOfflineMessages() and how changes to it might affect it's callers, and I didn't have the time or resources to confirm this on my project.

But these constraints don't apply here, so you're right of course, the check should be in canFloodOfflineMessages()

Guus der Kinderen
March 1, 2013, 9:49 PM

Simon, why did you choose to implement this as a check that is additional to the canFloodOfflineMessages() call, instead of as part of the implementation of canFloodOfflineMessages()?

Simon White
February 25, 2013, 10:11 AM

Simple .diff attached which seems to resolve this issue - when 'flooding' messages to a client which has just become online, only do so if priority > -1

Fixed

Assignee

csh

Reporter

LG