Details

    • Type: New Feature New Feature
    • Status: Not Yet Scheduled Not Yet Scheduled
    • Priority: Low Low
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: XEP Support
    • Labels:
      None

      Description

      It would be nice if ejabberd supported XEP-0198: Stream Management, for both c2s and s2s. I'd feel more secure knowing that stanzas will be resent if lost.

      Existing implementations:

      • Psi: Gsoc'10
      • Prosody: preliminary patch, not yet published

        Activity

        Hide
        Holger Weiß
        added a comment -

        I have an initial patch. If you'd like to test it:

        https://github.com/weiss/ejabberd

        This is the raw diff:

        https://github.com/weiss/ejabberd/compare/master...xep-0198.diff

        The patch currently implements stanza ACKs but not stream resumption. I'll look into that soon.

        I'd be very interested in any kind of feedback.

        Show
        Holger Weiß
        added a comment - I have an initial patch. If you'd like to test it: https://github.com/weiss/ejabberd This is the raw diff: https://github.com/weiss/ejabberd/compare/master...xep-0198.diff The patch currently implements stanza ACKs but not stream resumption. I'll look into that soon. I'd be very interested in any kind of feedback.
        Hide
        Holger Weiß
        added a comment -

        I've now pushed code that adds stream resumption support into my repository, so the XEP-0198 support is now expected to be pretty much complete (except that it's limited to c2s connections so far).

        Show
        Holger Weiß
        added a comment - I've now pushed code that adds stream resumption support into my repository , so the XEP-0198 support is now expected to be pretty much complete (except that it's limited to c2s connections so far).
        Hide
        Dennis Schridde
        added a comment -

        Great! I'm looking forward to see this in deployment, soon.

        Are you also working on S2S support?

        Show
        Dennis Schridde
        added a comment - Great! I'm looking forward to see this in deployment, soon. Are you also working on S2S support?
        Hide
        Holger Weiß
        added a comment -

        I might have a go at s2s support, but I'd like to get the c2s code production-ready, first.

        If anyone interested in this feature could have a look at the code and/or test it, that would help a lot

        Show
        Holger Weiß
        added a comment - I might have a go at s2s support, but I'd like to get the c2s code production-ready, first. If anyone interested in this feature could have a look at the code and/or test it, that would help a lot
        Hide
        Holger Weiß
        added a comment -

        I created a pull request, so I guess any discussion on my actual patch should go there:

        https://github.com/processone/ejabberd/pull/166

        Show
        Holger Weiß
        added a comment - I created a pull request, so I guess any discussion on my actual patch should go there: https://github.com/processone/ejabberd/pull/166

          People

          • Votes:
            26 Vote for this issue
            Watchers:
            22 Start watching this issue

            Dates

            • Created:
              Updated:
              Days since last comment:
              5 days ago

              Issue deployment