[s2s] Federation issue with talkonaut.com

Description

My server (weather.im) happily federates with talkonaut.com with Openfire 3.10.3. The beta of Openfire 4.0 fails to federate, here's the trace logged.

My 3.10.3 instance reports that the outbound connect is unsecured dialback authentication and the inbound connection is secure.

Environment

None

Activity

Show:

Guus der Kinderen January 5, 2016 at 10:03 PM

We test-ran the fix on ignite - works fine!

Guus der Kinderen January 5, 2016 at 8:22 PM

My earlier fix changed TLS failure handling on the connection used after a S2S connection was successfully established. In this issue, we're not getting to that point (the earlier fix itself is an improvement though, but not for the issue at hand).

A problem occurs before the S2S connection is established, on the connection used for dialback. The changes for add TLS support to Dialback, but won't allow for TLS to fail.

PR https://github.com/igniterealtime/Openfire/pull/489 allows Dialback to be retried after a TLS failure occurred, skipping TLS.

Daryl Herzmann December 17, 2015 at 5:20 PM

talkonaut.com kindly engaged us on this and noted the following stanza

Guus der Kinderen December 17, 2015 at 12:12 PM
Edited

https://github.com/igniterealtime/Openfire/pull/466 changes the event from an 'exceptional' handshake failure to a normal one. This should make logging less verbose.

This does not, however, fix the issue with S2S to talkanout.com - I'm guessing that new default settings for TLS now simply disallow federation.

Dave Cridland December 17, 2015 at 8:38 AM

TLS (and SSLv3 before it) include an Alert record which, amongst other things, indicates the session is being closed, in order to protect against a truncation attack where you kill a session (by spoofing TCP RST packets, etc) before all the data is across.

HTTP/1.1 is somewhat immune to this, and it has no effect at all on XMPP.

Swallowing this stack trace would be fine.

I note though that we're being a little over the top in validating certificates at the moment - I'll review that code over the next week or so and see if we can make it work a little closer to expectations.

Fixed

Details

Assignee

Reporter

Labels

Components

Fix versions

Affects versions

Priority

Created December 17, 2015 at 3:20 AM
Updated August 9, 2019 at 11:28 AM
Resolved January 5, 2016 at 10:03 PM