When anonymous user try to access a forbidden page, access is denied but wrong theme is used: garland (admin thme) in place of a funny custom 403 page !

CommentFileSizeAuthor
#23 theme403fix-759836.patch1015 byteskleinmp
#14 drupal6-759836.patch1.18 KBgclicon
#6 403themefix.patch1.09 KBwOOge
#4 403themefix.patch1.09 KBwOOge

Comments

dddave’s picture

dddave commented
Category: bug » support

I guess you changed the site path for this error (admin/settings/error-reporting)? I am pretty sure there is a handbooksection about customizing this page.

In the modules section are some modules for handling 403s. Might be worth looking through.

Anonymous’s picture

Anonymous (not verified) commented

Yes, I changed the site path.

I will follow your indications and look for some explanations.

Thanks

wOOge’s picture

wOOge commented
Version: 6.16 » 6.20
Category: support » bug

I'm having this same issue — 403 error pages, even if it's redirected to another page (having set it in admin/settings/error-reporting), the site admin theme is used, not the default site theme.

Apparently this problem was fixed in D7 via this patch:
http://drupal.org/node/346455

Could this be back-ported?

wOOge’s picture

wOOge commented
Status: Active » Needs review
StatusFileSize
new 403themefix.patch1.09 KB

... ok I have back ported the D7 patch from this post ( http://drupal.org/node/346455 ) and tested on D6 6.20 codebase and seems to work.

Can someone verify that this works and commit the patch?

Status: Needs review » Needs work

The last submitted patch, 403themefix.patch, failed testing.

wOOge’s picture

wOOge commented
Status: Needs work » Needs review
StatusFileSize
new 403themefix.patch1.09 KB

Status: Needs review » Needs work

The last submitted patch, 403themefix.patch, failed testing.

wOOge’s picture

wOOge commented

Patch keeps failing... not sure what I'm doing wrong —
I followed these instructions for using DIFF: http://drupal.org/patch/create

dddave’s picture

dddave commented

Patches are failing due to: http://drupal.org/node/961172

wOOge’s picture

wOOge commented
Status: Needs work » Needs review

#4: 403themefix.patch queued for re-testing.

wOOge’s picture

wOOge commented

Patches are failing due to: http://drupal.org/node/961172

I'm not sure if this thread is telling me there is something I can do to my patch, or if the system for testing patches has a problem.

Sorry... seems I'm patch inept :(

Anonymous’s picture

Anonymous (not verified) commented

#6: 403themefix.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, 403themefix.patch, failed testing.

gclicon’s picture

gclicon commented
Status: Needs work » Needs review
StatusFileSize
new drupal6-759836.patch1.18 KB

I tested your patch and it did the trick. After reviewing the details on the failed test on the patch, I have updated the patch. Hopefully this version passes the testing.

Status: Needs review » Needs work

The last submitted patch, drupal6-759836.patch, failed testing.

gclicon’s picture

gclicon commented

Looks like it didn't pass the test either, but the patch works fine.

dddave’s picture

dddave commented

Please visit the issue mentioned in #9 to see why the patch failed. While you are there you could provide some feedback to get the patch committed.

ducktape’s picture

ducktape commented
Status: Needs work » Needs review

#14: drupal6-759836.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal6-759836.patch, failed testing.

ducktape’s picture

ducktape commented

Hm, I thought the patch failing problem was fixed here http://drupal.org/node/961172

Randy mentioned named patches, but according to this thread http://drupal.org/node/973432, patches should not be named to be applicable.

hacmx’s picture

hacmx commented
Status: Needs work » Needs review

#14: drupal6-759836.patch queued for re-testing.

Status: Needs review » Needs work

The last submitted patch, drupal6-759836.patch, failed testing.

kleinmp’s picture

kleinmp commented
Version: 6.20 » 6.x-dev
Status: Needs work » Needs review
StatusFileSize
new theme403fix-759836.patch1015 bytes

Here's the patch updated to be done off of git.

steinmb’s picture

steinmb commented

Looks good to me, anything that is holding up this small patch?

Status: Needs review » Closed (outdated)

Automatically closed because Drupal 6 is no longer supported. If the issue verifiably applies to later versions, please reopen with details and update the version.