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, 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, 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

Assignee

Gaston Dombiak

Reporter

LG