When LocalOutgoingServerSession.createOutgoingSession encounters an error in connecting it logs a verbose error to the logs including a stack trace to the exception site. Seeing most of these error types are expected (such as host not found) and the stack trace is not especially useful in solving these problems this is needlessly verbose and makes it harder for a server operator to find real problems. This is componeded by openfire then traversing the domain to try connect.
Suggestion: expected errors like HostNotFound and Connection timeout errors should just log the information, other exceptions should continue to log the full error.
I'm OK with this; will merge and validate.
Modified patch for more recent changes to LocalOutgoingServerSession.
Reverted patch because it broke the TLS handling logic for S2S connections.
Running igniterealtime with updated patch, TLS s2s sessions looks good now. I think we can close this
Updated to suppress stack traces only, without changing S2S exception handling logic.