Closed (cannot reproduce)
Project:
Redirect 403 to User Login
Version:
6.x-1.2
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
4 Oct 2010 at 17:41 UTC
Updated:
12 Nov 2010 at 08:31 UTC
Jump to comment: Most recent file
Comments
Comment #1
damienmckennaAm seeing this too.
Comment #2
damienmckennaThis is partly a bug in the drupal_goto() function:
The key part is the if($destination) line, which will ignore the "?destination=" query string.
What r4032login needs to do is instead add "?destination=", which does work. Stay tuned for a patch.
Comment #3
damienmckennaHere are patches for the DRUPAL-6--1 and DRUPAL-7--1 branches that fix the problem.
Comment #4
deekayen commentedCommitted to DRUPAL-6--1, DRUPAL-7--1, and HEAD.
Comment #5
benjarlett commentedCommitted to DRUPAL-6--1, DRUPAL-7--1, and HEAD.
Forgive me if I've missed something, but I've updated this module to latest version and still get this error.. do I have to patch it... I have patch fear.
Thanks.
Comment #6
deekayen commentedAre you sure you downloaded the 6.x-1.x-dev release? http://drupal.org/node/316860
I like them to sit in the development release for a couple days before I release a new, tagged stable version.
Comment #7
deekayen commentedI changed the implementation of #3 in DRUPAL-7--1 because $_REQUEST seems to be no longer available and ?q= has a value of r4032login.
Comment #8
deekayen commentedComment #9
thibHi,
The patch works for me (D6)
Thanks,
Thib