Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
This happens on Drupal.org as well. Normally when you go to a non-existent node, you get a 404 error page, either the default or custom if you have set one up. What happens when you go to a non existent node that shares the first digits with a container is another matter. When you do this, you get an empty forum. So if the path to a container is: /node/615 , any node that begins with the numbers 615 will come up with an empty forum instead of the 404 page it should display.
node/6152
node/6157
node/61567399458
all these will show an empty forum.
Comment | File | Size | Author |
---|---|---|---|
#3 | forum-send-404-instead-of-generating-empty-pages.patch | 981 bytes | Bart Jansens |
Comments
Comment #1
Liam McDermott CreditAttribution: Liam McDermott commentedI wasn't able to reproduce this with Drupal 5 (I get a 404 error, as expected). Have you tried this on a new install of Drupal with no contributed modules? It could be some wayward module that's causing the problem.
Comment #2
diabloview CreditAttribution: diabloview commentedI have tried it on a fresh install. It happens on this very web site too, and Gallery 2's web site. I added all the 5's after clicking on a forum container. Try these links to see what you get, it's an empty forum, not a 404 page.
gallery.menalto.com/forum/1155555555
drupal.org/forum/85555555555555
Comment #3
Bart Jansens CreditAttribution: Bart Jansens commentedI think the URLs in the original example are wrong. Forum module generates empty forum pages for anything below forum/. This is indeed a bug. This happens a lot if you use pathauto and then delete a node.
Attached is a patch which i am using on my site. But this problem seems to exist in D6 as well, so maybe we should have it fixed in head first and then backport?
Comment #4
keith.smith CreditAttribution: keith.smith commentedSmall coding style issue: comments should begin with a capital letter, and end in a period.
Comment #5
MichelleBumping the version. If it's still a problem, the patch probably needs a reroll.
Michelle
Comment #6
gregglesDamZ has a fix for this in the patch in #520736: SA-CORE-2009-007 forum module XSS. I'm tempted to mark this a duplicate of that but based on my last comment there the reverse might make more sense...