We're updating the issue view to help you get more done. 

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

Acceptance Test - Entry

None

Activity

Show:
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

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.

Assignee

Gaston Dombiak

Reporter

LG

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Major
Configure