Fixing errors in mod_roster

esl · Nov 24, 2022 · d8b17a9 · d8b17a9
1 parent d5abdd6
commit d8b17a9
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/priv/graphql/schemas/admin/roster.gql b/priv/graphql/schemas/admin/roster.gql
@@ -9,7 +9,7 @@ type RosterAdminMutation @protected{
   addContacts(user: JID!, contacts: [ContactInput!]!) : [String]!
     @protected(type: DOMAIN, args: ["user"])
   "Manage the user's subscription to the contact"
-  subscription(user: JID!, contact: JID!, action: SubAction): String
+  subscription(user: JID!, contact: JID!, action: SubAction!): String
     @protected(type: DOMAIN, args: ["user"])
   "Delete user's contact"
   deleteContact(user: JID!, contact: JID!): String

diff --git a/priv/graphql/schemas/user/roster.gql b/priv/graphql/schemas/user/roster.gql
@@ -19,7 +19,7 @@ Allow user to get information about user roster/contacts.
 """
 type RosterUserQuery @protected{
   "Get the user's roster/contacts"
-  listContacts: [Contact!] 
+  listContacts: [Contact!]
   "Get the user's contact"
   getContact(contact: JID!): Contact
 }
diff --git a/src/mod_roster_api.erl b/src/mod_roster_api.erl
@@ -107,7 +107,7 @@ subscription(#jid{lserver = LServer} = CallerJID, ContactJID, Type) ->
                                        lserver => LServer,
                                        element => El }),
             Acc2 = mongoose_hooks:roster_out_subscription(Acc1, CallerJID, ContactJID, Type),
-            ejabberd_router:route(CallerJID, ContactJID, Acc2),
+            ejabberd_router:route(CallerJID, jid:to_bare(ContactJID), Acc2),
             {ok, io_lib:format("Subscription stanza with type ~s sent successfully", [StanzaType])};
         {error, not_found} ->
             ?UNKNOWN_DOMAIN_RESULT