gsitemap lists sites which are not accessible for an anonymous user...
this shouldnt be...

I attached a small patch which excludes this nodes.
But be warned: I dont know how much time this costs...
This was just a fast solution I dont know if it is the best...

CommentFileSizeAuthor
no_access.patch.txt975 bytesTobias Maier

Comments

Tobias Maier’s picture

Tobias Maier commented
Status: Active » Needs review
twohills’s picture

twohills commented

I came up with exactly the same hack :-) Note you have a typo in your patch: spelling of "continue".
This doesn't seem to work for all node types, e.g project issues are slipping through. I'm still digging to find out why...

SamAMac’s picture

SamAMac commented
Assigned: Unassigned » SamAMac
Status: Needs review » Postponed

Hmmm. My guess is that certain things don't properly implement hook_sql, so the db_rewrite_sql call doesn't exclude them.

I will do some investigating of these other modules, but I'd rather avoid anything that would increase the running time if I can avoid it. There are already problems with exceeding maximum execution time on large sites.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

Set issue 132503 to duplicate of this issue.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Postponed » Postponed (maintainer needs more info)

I enabled organic groups access control, logged out, and loaded the site map. Nodes which were set to be viewed only by group members were not included. This was with the code from the DRUPAL-5 branch. I wonder, have the people who are complaining enabled access control and logged out before viewing the site map?

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Postponed (maintainer needs more info) » Closed (duplicate)

Duplicate of issue 140291.