- 21 Mar, 2016 4 commits
-
-
Guus der Kinderen authored
Denote Openfire 4.0.2 Release
-
akrherz authored
-
akrherz authored
-
akrherz authored
Since the release of Openfire 3.9.3, users have reported problems with "ghost" sessions left on the server. After reviewing the changes that went into the 3.9.3 release, it seemed the fix for OF-464 may have had some side effect causing this. Since Igniterealtime's Openfire was reproducing OF-829, I did a test whereby the functional changes of Igniterealtime/Openfire#ad08cae9 were reverted. After 36 hours, there were no ghosts! This will likely necessitate reopening OF-464.
-
- 17 Mar, 2016 12 commits
-
-
Guus der Kinderen authored
IQ result should be the last stanza. This way, clients know that the last archived message has arrived.
-
wroot authored
-
Guus der Kinderen authored
Some user providers might not provide a creation date for each user. There's no reason that the Admin Console should not be able to cope with that.
-
Chinmay Dani authored
With the current implementation, any user can send the register IQ to register with a room as a 'member'. This also allows outcasts, that were banned from a room, to be able to register again as a member, and eventually re-join the room. The solution is to prevent outcasts from registering with the room.
-
Guus der Kinderen authored
In dom4j, the default namespace of an element cannot be set by adding a namespace that has an empty-string prefix. Although the string representation of the element will be correct, that namespace is not being recognized as the default namespace of the element (instead, it is a namespace like any other prefixed namespace, to be used in child elements).
-
Guus der Kinderen authored
The original code depends on a whitespace to be present within the limits of the element. This is not safe (eg: <message><foo/></message> fails). Instead, the closing characters ( > or /> ) of the first element should be used, as that's guaranteed to be present.
-
Guus der Kinderen authored
A change introduced in Openfire 3.10.3 (OF-938) changed how the BOSH implementation would add default namespace definitions where applicable. This new implementation depends on a space from being present in the outer stanza. The 'fin' element that's transmitted here is wrapped in a message stanza without any attributes ( <message><fin ... ) which caused the namespace to be injected in the fin element (where the first space was). This commit adds a 'to' attribute to the wrapping message stanza, to allow for users to work around the problem without having to update Openfire itself. The BOSH implementation will be adjusted to account for this in another commit.
-
Derek McLean authored
When a room occupant sends an availability update to a host in an Openfire cluster, other users cannot join the room from other hosts in the cluster. The availability update causes the other hosts to lose an occupant's role and affiliation. New occupants encounter an NPE when joining and are prevented from joining the room. Specifically, the NPE occurs when Openfire attempts to send initial presences for current occupants. Remote hosts in the cluster lose the occupant's association because the local room simply broadcasts the presence packet for the occupant's availability update. Because it is from the client, this packet does not have the occupant's role or affiliation. The remote hosts treat the presence packet as the occupant's presence without modification. This fix changes the order in which Openfire handles an availability update. First, it updates its local view of an occupant's presence. This populates the correct association. Then, it broadcasts the updated presence to remote hosts in the cluster.
-
Chung authored
In the current read scheme for Openfire, it is possible that due to variable width encoding that we sometimes only create a string with a partial character. This breaks the message text and causes encoding issues when read. This change modifies the way the characters are read to use an bytearrayoutputstream and only do the string conversion once. Since we only do the string creation once after the full buffer is read, we no longer have encoding issues with broken unicode characters.
-
dchinmay authored
In #routeToComponent, it is first checked if the component is being hosted in the local JVM. If it was not found, then all nodes in the ComponentsCache are checked to find if the component is hosted by one of those. In this process, the local routing table is again searched for the component, because it may have been added after the previous check. However, after this search, there is no null check done for the searched route. This results in an NPE when the component was not found locally and the #process(packet) method is called on the object. This also prevents the code from searching other nodes that may have been in the nodes set. Added a null check to prevent this scenario.
-
Christian Schudt authored
See XEP-0016, example 50.
-
Tom Evans authored
Patch submitted by community member lukzie.
-
- 26 Jan, 2016 4 commits
-
-
Guus der Kinderen authored
-
Guus der Kinderen authored
-
Guus der Kinderen authored
This commit moves jetty 9.2.9 to 9.2.14, which currently is the latest release of it 9.2 branch. Conflicts: build/lib/versions.txt
-
Tom Evans authored
Ensure cluster listener is properly initialized (and destroyed).
-
- 25 Jan, 2016 16 commits
-
-
Christian Schudt authored
-
Guus der Kinderen authored
The unmodifiable collection in the original code is not immutable: when the original collection is modified, this is reflected (which can cause problems in iterators created from the unmodifiable collection). This commit creates a new collection to prevent this from happening.
-
André Berenguel authored
-
André Berenguel authored
-
André Berenguel authored
-
André Berenguel authored
-
André Berenguel authored
-
André Berenguel authored
Conflicts: src/java/org/jivesoftware/openfire/keystore/IdentityStore.java
-
André Berenguel authored
-
Guus der Kinderen authored
-
André Berenguel authored
IdentityStore.addSelfSignedDomainCertificate considers the certificate validity in days, not in months.
-
Guus der Kinderen authored
The admin console should existing-but-invalid configuration. Configuration should be modifiable through admin console.
-
Guus der Kinderen authored
By reversing the order in which objects are added to the internal state, a failure during store construction won't put the manager in a half-initialized state.
-
Guus der Kinderen authored
Check if the keys in the identity store can be accessed when the identity store is being created.
-
Guus der Kinderen authored
Don't initialize the admin console TLS connector when no identity store is available (that's guaranteed to fail). Don't log admin console startup success when there's an error.
-
wroot authored
-
- 21 Jan, 2016 2 commits
-
-
Guus der Kinderen authored
Replaced all apostrophes with HTML-encoded entities to prevent formatting problems.
-
Guus der Kinderen authored
Existing code already tried to prevent message bounces when a stanza was sent by the server itself. This commit builds on that solution. Also, debug logging was added for the store-and-* strategies when storage was no longer possible as a result of quota limitations (we had a hard time debugging this)
-
- 19 Jan, 2016 1 commit
-
-
Nik Okuntseff authored
Made a search by distinguishedName Active Directory specific as it is difficult to come up with a generic solution.
-
- 18 Jan, 2016 1 commit
-
-
Christian Schudt authored
The problem was, that the logic stored empty chat messages, which contained a thread and chat state. The loop first checked the namespace of the thread, which was empty and therefore thought it should store it.
-