Package home | Report new bug | New search | Development Roadmap Status: Open | Feedback | All | Closed Since Version 0.16.14

Request #2678 Seperate redirect URI for idle/expiration
Submitted: 2004-11-03 12:34 UTC
From: roehr at zilleon dot com Assigned: arnaud
Status: Closed Package: LiveUser
PHP Version: Irrelevant OS: irrelevant
Roadmaps: (Not assigned)    
Subscription  
Comments Add Comment Add patch


Anyone can comment on a bug. Have a simpler test case? Does it work for you on a different platform? Let us know! Just going to say 'Me too!'? Don't clutter the database with that please !
Your email address:
MUST BE VALID
Solve the problem : 43 + 25 = ?

 
 [2004-11-03 12:34 UTC] roehr at zilleon dot com
Description: ------------ I would like to request the inclusion of a way to distinguish between lgout redirects and auth idle/expire redirects. This could be achieved by adding an option to the config in the authContainers array: 'authContainers' => array(0 => array(redirect = '/index.php?error=timeout')) Lukas suggested this alternative: Maybe we could define a standard for placeholders in the redirect url and we would replace all those placeholders with all the relevant properties. That way we would not have to add another option. It would however mean running a bunch if substr_replace() on every redirect. Thanks for having a look! Torsten Roehr

Comments

 [2004-11-03 12:37 UTC] smith at backendmedia dot com
We have a new plan. We want to remove the redirect and callback config parameters in favor of a more general observer pattern. This allows you to register observers so that they get notified of any events. This should allow anything you want and it will remove some of the config clutter we currently have.
 [2004-12-08 20:03 UTC] arnaud
This bug has been fixed in CVS. In case this was a documentation problem, the fix will show up at the end of next Sunday (CET) on pear.php.net. In case this was a pear.php.net website problem, the change will show up on the website in short time. Thank you for the report, and for helping us make PEAR better. The observer patch from Matthias was committed, it still needs testing and documentation. It enables a broad range of possibilities.