Hi,
I am testing this module. Thanks for building this!
A lot works, but this pages: /admin/reports/security-review/help/views/access gives 404.

greetings, Martijn

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

coltrane’s picture

Status: Active » Postponed (maintainer needs more info)

Do you have the Views module enabled?

tangent’s picture

I am also seeing this issue. There are 2 checks for which the "details" link does not work and those URLs have a different pattern from the others. I am using Views 6.x-3.x.

/admin/reports/security-review/help/security_review/admin_permissions (WORKS)
/admin/reports/security-review/help/filefield/filefield_extensions (DOES NOT WORK)
/admin/reports/security-review/help/views/access (DOES NOT WORK)

I checked to see if adding "security_review" to the URL, to match the pattern of other URLs, would work but it does not.

Looking at the code, the filefield and views checks are handled differently than other checks. Perhaps the way the checks arrays are merged is responsible. I haven't identified the fix yet though.

webservant316’s picture

same problem here.

webservant316’s picture

I just tried /admin/reports/security-review/help/views/access as both an admin user and user 1 and both result in a 404.

coltrane’s picture

@tangent @webservant316 can you both confirm you have the Views module installed and enabled?

tangent’s picture

As previously mentioned, I am using Views 6.x-3.x.

webservant316’s picture

I am using Views 6.x-2.16

Justin_KleinKeane’s picture

Status: Postponed (maintainer needs more info) » Needs review
FileSize
1.79 KB

I can confirm that this error occurs even when Views and Views Access modules are enabled and installed.

It seems like there is an issue in the security_review_check_help function, mainly because when the view access help link is clicked (from ?q=admin/reports/security-review) the result it ?q=admin/reports/security-review/help/views/access, so the $module in security_review_check_help is 'views' and the $check_name is 'access'. This causes the if (function_exists) conditional to fail since 'views_security_checks' is not, in fact, a defined function. The module seems to expect that the $module will always be 'security_review', and if it is this function works just fine (even with views help). I wrote the following patch to remove the conditional and simply use the expected function: security_review_security_checks(). Patch is attached.

coltrane’s picture

I'm having a hard time understanding this issue, sorry. It sounds straight forward but I have yet to reproduce. @Justin_KleinKeane can you roll your patch against 6.x-1.x please?

Will one of you test against 6.x-1.x dev and report if you still receive this error. Also, list of enabled modules would help, specifically I don't understand if "Views Access" is a module or if this error is occurring with just the Views module enabled.

coltrane’s picture

I was able to replicate this with 6.x-1.2 but I cannot with 6.x-1.x-dev. I'm planning a new release of 6.x-1.x soon.

Status: Needs review » Needs work

The last submitted patch, security_review-6.x-1.2_views_access_help.patch, failed testing.

mr.j’s picture

Issue summary: View changes

This is still a problem in the current release.

BigMike’s picture

Also having this issue. v6.x-1.2.
/admin/reports/security-review/help/filefield/filefield_extensions (page not found)
/admin/reports/security-review/help/views/access (page not found)

It's going to be another month for us to finish our migration to D7 so I figured I'd revisit our security review. Found this issue, searched, figured I'd post up.

The D6 branch is probably dead anyhow. Just passing through :)

dsnopek’s picture

Status: Needs work » Closed (duplicate)
Related issues: +#1361640: Unable to access filefield or views reports

This is a duplicate of #1361640: Unable to access filefield or views reports which is included in 1.3 release! This is fixed.