In testing Xapian search results on KernelTrap, I found it was pointing me to unpublished nodes. Furthermore, in looking at the code I believe if there are multiple versions of the same node, it will index both versions. At minimum, the query in xapian_reindex_all_confirm_submit() needs to only insert published nodes into the xapian_index_queue.

Comments

jeremy’s picture

jeremy commented
Assigned: Unassigned » jeremy

Committed a patch so we no longer index unpublished content. Leaving issue open, as there's still the issue of unpublished content that was already indexed.

jeremy’s picture

jeremy commented
Status: Active » Fixed

Committed another patch to un-index previously indexed unpublished content (on KernelTrap this is generally spam).

Anonymous’s picture

Anonymous (not verified) commented
Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for two weeks with no activity.