Uploaded image for project: 'Smack'
  1. Smack
  2. SMACK-711

Improve the logging of TCP connection attempts.

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Resolved
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 4.1.5
    • Fix Version/s: 4.2.0
    • Component/s: TCP
    • Labels:
      None

      Description

      XMPPTCPConnection using ConnectionException currently does not provide the individual IPv(4|6) addresses it tried to connect to and failed. It only logs those in https://github.com/igniterealtime/Smack/blob/f79a7d9d5f3bd4444fda7e85426495d3fa097321/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java#L561 and https://github.com/igniterealtime/Smack/blob/f79a7d9d5f3bd4444fda7e85426495d3fa097321/smack-tcp/src/main/java/org/jivesoftware/smack/tcp/XMPPTCPConnection.java#L576. This makes it hard to users to determine the exact reason why a connection attempt failed in case the client is IPv6 only and the XMPP services announces availability via IPv6 but does not actually listen on an IPv6 address. See for example https://github.com/redsolution/xabber-android/issues/539

      Basically Smack's HostAddress needs to be changed so that it allows holding multiple exceptions.

        Attachments

          Activity

            People

            Assignee:
            flow Florian Schmaus
            Reporter:
            flow Florian Schmaus
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: