# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1403800393 -7200
# Node ID 2cd667d8edbd29b419c41a4d144269002680de77
# Parent  13e359c48b5bd95a49e6a94f621ce741b9f544c2
mod_pubsub_feeds: Handle denied subscription requests

diff -r 13e359c48b5b -r 2cd667d8edbd mod_pubsub_feeds/mod_pubsub_feeds.lua
--- a/mod_pubsub_feeds/mod_pubsub_feeds.lua	Thu Jun 26 18:32:43 2014 +0200
+++ b/mod_pubsub_feeds/mod_pubsub_feeds.lua	Thu Jun 26 18:33:13 2014 +0200
@@ -208,13 +208,15 @@
 				module:log("debug", "Invalid topic: %s", tostring(query["hub.topic"]))
 				return 404
 			end
-			if query["hub.mode"] ~= feed.subscription then
+			if query["hub.mode"] == "denied" then
+				module:log("info", "Subscription denied: %s", tostring(query["hub.reason"] or "No reason given"))
+				feed.subscription = "denied";
+				return "Ok then :(";
+			elseif query["hub.mode"] == feed.subscription then
+				module:log("debug", "Confirming %s request to %s", feed.subscription, feed.url)
+			else
 				module:log("debug", "Invalid mode: %s", tostring(query["hub.mode"]))
 				return 400
-				-- Would this work for unsubscribe?
-				-- Also, if feed.subscription is changed here,
-				-- it would probably invalidate the subscription
-				-- when/if the hub asks if it should be renewed
 			end
 			local lease_seconds = tonumber(query["hub.lease_seconds"]);
 			if lease_seconds then