From f33026a3f80b7ed827a0539c65b44a290dbc7424 Mon Sep 17 00:00:00 2001 From: Jeffrey Hutzelman Date: Thu, 7 Feb 2013 16:43:46 -0500 Subject: server: Send SERVACK in response to GIMMESUBS When processing a request to retrieve subscriptions, we actually do need to send a SERVACK in response to the incoming notice before sending the subscription list (directly) to the client that requested it. The line that did this was inadvertently removed when the OLD_COMPAT support was removed in commit 9b709859db5310444052d13ed8ebccec6ead1669. This fixes #91 --- server/dispatch.c | 1 + 1 file changed, 1 insertion(+) diff --git a/server/dispatch.c b/server/dispatch.c index 8b9507a..256a786 100644 --- a/server/dispatch.c +++ b/server/dispatch.c @@ -990,6 +990,7 @@ control_dispatch(ZNotice_t *notice, someone who has no subscriptions does NOT get a SERVNAK but rather an empty list. Note we must therefore check authentication inside subscr_sendlist */ + ack(notice, who); subscr_sendlist(notice, auth, who); return ZERR_NONE; } else if (!auth) { -- cgit v1.2.3