Changeset

5574:10b7118bbd8f

mod_pubsub_feeds: Remove broken attempt to generate an ID from content This seems to never have worked correctly and now the timestamp is out of scope anyway.
author Kim Alvefur <zash@zash.se>
date Sun, 25 Jun 2023 16:45:25 +0200
parents 5573:ed177fa5b54c
children 5575:372e59bd45d7
files mod_pubsub_feeds/mod_pubsub_feeds.lua
diffstat 1 files changed, 2 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/mod_pubsub_feeds/mod_pubsub_feeds.lua	Sun Jun 25 16:42:57 2023 +0200
+++ b/mod_pubsub_feeds/mod_pubsub_feeds.lua	Sun Jun 25 16:45:25 2023 +0200
@@ -109,8 +109,8 @@
 				module:log("debug", "Feed %q item %s is missing an id, using <link> instead", item.url, entry:top_tag());
 				id = link and link.attr.href;
 			else
-				module:log("debug", "Feed %q item %s is missing an id, using a HMAC of the item instead", item.url, entry:top_tag());
-				id = item.url .. "#" .. hmac_sha1(item.url, tostring(entry), true) .. "@" .. dt_datetime(timestamp);
+				module:log("error", "Feed %q item %s is missing both id and link, this feed is unusable", item.url, entry:top_tag());
+				return;
 			end
 			entry:text_tag("id", id);
 		end