Comparison

MUC: Catch and log error in case of storage failure in iterator over rooms

comparison

equal deleted inserted replaced

-:7fc0784491ef
+:a4e63b037a2a
 		local seen = {}; -- Don't iterate over persistent rooms twice
 		for room in rooms:values() do
 			coroutine.yield(room);
 			seen[room.jid] = true;
 		end
-		for room_jid in pairs(persistent_rooms_storage:get(nil) or {}) do
+		local all_persistent_rooms, err = persistent_rooms_storage:get(nil);
+		if not all_persistent_rooms then
+			if err then
+				module:log("error", "Error loading list of persistent rooms, only rooms live in memory were iterated over");
+				module:log("debug", "%s", debug.traceback(err));
+			end
+			return nil;
+		end
+		for room_jid in pairs(all_persistent_rooms) do
 			if not seen[room_jid] then
 				local room = restore_room(room_jid);
 				if room then
 					coroutine.yield(room);
 				else