authorization-identifier is a sasl2 feature, not bind

Author: sonnyp

packages/client-core/src/bind2/bind2.js

@@ -2,7 +2,7 @@ import xml from "@xmpp/xml";
 
 const NS_BIND = "urn:xmpp:bind:0";
 
-export default function bind2({ sasl2, entity }, tag) {
+export default function bind2({ sasl2 }, tag) {
   const features = new Map();
 
   sasl2.use(
@@ -22,9 +22,6 @@ export default function bind2({ sasl2, entity }, tag) {
       );
     },
     (element) => {
-      const aid = element.parent.getChildText("authorization-identifier");
-      if (aid) entity._jid(aid);
-
       for (const child of element.getChildElements()) {
         const feature = features.get(child.getNS());
         if (!feature?.[1]) continue;

packages/client/index.js

@@ -62,7 +62,7 @@ function client(options = {}) {
   );
 
   // SASL2 inline features
-  const bind2 = _bind2({ sasl2, entity }, resource);
+  const bind2 = _bind2({ sasl2 }, resource);
 
   // Stream features - order matters and define priority
   const sasl = _sasl(

packages/sasl2/index.js

@@ -71,6 +71,9 @@ async function authenticate({
           mech.final(decode(additionalData));
         }
 
+        const aid = element.getChildText("authorization-identifier");
+        if (aid) entity._jid(aid);
+
         for (const child of element.getChildElements()) {
           const feature = features.get(child.getNS());
           if (!feature?.[1]) continue;