Uploaded image for project: 'ejabberd development'
  1. ejabberd development
  2. EJAB-1045

Routes vCard request to the occupant full JID, but should to bare JID

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: ejabberd 2.0.5, ejabberd 2.1.0
    • Component/s: vCard
    • Labels:
      None
    • Environment:
      all

      Description

      ejabberd intercepts vCard requests sent to full JIDs. The RFC requires they be forwarded to the client.

        Issue Links

          Activity

          Hide
          neustradamus Neustradamus added a comment - - edited

          but it is not hard to correct this bug, no ?

          Show
          neustradamus Neustradamus added a comment - - edited but it is not hard to correct this bug, no ?
          Hide
          badlop Badlop added a comment -

          1045-21-fixiqrouting.diff implements this:

          "The server SHOULD deliver the stanza to that connected resource"

          And 1045-21-mucvcardbarejid.diff fixes this this:

          mod_muc is sending vcard requests to the full JID, rather than the bare JID

          Show
          badlop Badlop added a comment - 1045-21-fixiqrouting.diff implements this: "The server SHOULD deliver the stanza to that connected resource" And 1045-21-mucvcardbarejid.diff fixes this this: mod_muc is sending vcard requests to the full JID, rather than the bare JID
          Hide
          louiz louiz' added a comment -

          I just tested both patch (on a 2.1.0), and it seems to fix the issue. It works as expected.
          Thank you

          Show
          louiz louiz' added a comment - I just tested both patch (on a 2.1.0), and it seems to fix the issue. It works as expected. Thank you
          Hide
          badlop Badlop added a comment -

          The patch 1045-21-fixiqrouting.diff included a change that will be applied in ejabberd only when bis-9 obsoletes RFC 3920.

          So that code is moved to the ticket EJAB-1113, and the patch is reduced accordingly.

          Show
          badlop Badlop added a comment - The patch 1045-21-fixiqrouting.diff included a change that will be applied in ejabberd only when bis-9 obsoletes RFC 3920. So that code is moved to the ticket EJAB-1113 , and the patch is reduced accordingly.
          Hide
          badlop Badlop added a comment -

          The two patches are committed to 2.0.x branch and trunk in svn r2766 .. 2770.

          Show
          badlop Badlop added a comment - The two patches are committed to 2.0.x branch and trunk in svn r2766 .. 2770.

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development