manager.xml is not using the correct property for setting the client port

Description

ConnectionManager uses the property xmpp.socket.plain.port to know which port to use for c2s communication. However, manager.xml is using xmpp.socket.plain for the same purpose. We need to update manager.xml to use the correct property.

Environment

None

Activity

Show:

Gaston Dombiak August 3, 2006 at 4:07 AM

Hey rouven,

Actual fix is to change manager.xml and keep using the same property. The reason for that is that we want to keep plain and ssl as close as possible.

Rouven Homann August 2, 2006 at 8:03 PM

ok, fixed the bug...

Index: F:/Workspace/Wildfire Connection Manager/src/java/org/jivesoftware/multiplexer/ConnectionManager.java
===================================================================
— F:/Workspace/Wildfire Connection Manager/src/java/org/jivesoftware/multiplexer/ConnectionManager.java (revision 4765)
+++ F:/Workspace/Wildfire Connection Manager/src/java/org/jivesoftware/multiplexer/ConnectionManager.java (working copy)
@@ -251,7 +251,7 @@

private void startClientListeners(String localIPAddress) {
// Start clients plain socket unless it's been disabled.

  • int port = JiveGlobals.getIntProperty("xmpp.socket.plain.port", 5222);
    + int port = JiveGlobals.getIntProperty("xmpp.socket.plain", 5222);
    ServerPort serverPort = new ServerPort(port, serverName, localIPAddress,
    false, null, ServerPort.Type.client);
    try {

cheers,

rouven

Fixed

Details

Assignee

Reporter

Fix versions

Affects versions

Priority

Created July 26, 2006 at 5:15 AM
Updated August 3, 2006 at 4:08 AM
Resolved August 3, 2006 at 4:08 AM

Flag notifications