Not Respecting Node Access

sami_k - May 4, 2006 - 05:59
Project:Atom
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:critical
Assigned:Dave Reid
Status:closed
Description

This module isn't respecting node access in that its making titles available of nodes not available to anonymous users.

#1

Christefano - October 27, 2006 - 20:14

I can confirm this bug. I'll look into fixing this and will post again if I have something.

#2

deekayen - March 1, 2007 - 05:01
Priority:normal» critical

Marking this critical because I think it should be fixed before any of the branches are tagged to be 1.0 stable.

#3

Christefano - March 1, 2007 - 14:59

Agreed, and thanks.

#4

deekayen - March 2, 2007 - 05:14

Can someone confirm the same problem happening with the core RSS feed? The reason I ask is Atom and RSS have the same query to the node table.

#5

Christefano - March 10, 2007 - 02:25

You're right. It looks like any promoted node that's published to the core rss.xml feed is visible, even to unprivileged users.

#6

deekayen - March 10, 2007 - 02:34

So... now what? 1) File a bug against core RSS, 2) "won't fix" this as a copy of core function, or 3) add node access as a feature? Is there a 4)? I don't have private content published to the front page on any of my sites, so from my more distant point of view, I'm leaning towards 1&3.

#7

deekayen - March 10, 2007 - 15:54

I'm not able to reproduce this so far. Could I get a step by step? What node types are viewable in the feed to unprivileged users? By unprivileged, I've been assuming you mean the "access content" right in the node content area of the access control page. What version of Drupal?

#8

deekayen - December 5, 2007 - 15:01
Status:active» fixed

committed a fix to DRUPAL-4-6, DRUPAL-4-7, DRUPAL-5, DRUPAL-6-1, and HEAD.

#9

Anonymous - December 19, 2007 - 15:02
Status:fixed» closed

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

#10

deekayen - September 21, 2009 - 01:21
Status:closed» needs work

1.22.2.1 is where the node_access was added for a quick workaround. It really ought to have a proper node_access implementation with rewrites. Re-opened because of #458034: Not yet moderated node found in atom feed! (modr8 or atom issue?).

#11

Dave Reid - September 21, 2009 - 01:44
Version:4.7.x-1.x-dev» 6.x-1.x-dev

Again posting patch here.

AttachmentSize
458034-atom-dbrewritesql-D6.patch 3.88 KB

#12

Dave Reid - September 21, 2009 - 01:45
Status:needs work» needs review

#13

Dave Reid - September 21, 2009 - 18:58
Assigned to:Anonymous» Dave Reid
Status:needs review» fixed

Ton of love has gone into the code recently. This has been fixed in the Drupal 6 branch, working on the DBTNG queries in the HEAD branch. Just going to mark this as fixed for now.

#14

System Message - October 5, 2009 - 19:00
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.