| Summary | redirect.php AUTH_REASON_FAILED |
| Queue | IMP |
| Queue Version | 4.1.1 |
| Type | Bug |
| State | Resolved |
| Priority | 1. Low |
| Owners | |
| Requester | lalot (at) univ-aix (dot) fr |
| Created | 04/21/2006 (7053 days ago) |
| Due | |
| Updated | 04/23/2006 (7051 days ago) |
| Assigned | |
| Resolved | 04/23/2006 (7051 days ago) |
| Github Issue Link | |
| Github Pull Request | |
| Milestone | |
| Patch | No |
State ⇒ Resolved
State ⇒ Unconfirmed
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ redirect.php AUTH_REASON_FAILED
Queue ⇒ IMP
a stange difference in the way redirect.php behave between 4.0 and 4.1.
I believe there is a mistake:
line 87:
if (($imapuser !== null && ($imapuser == $_SESSION['imp']['user'])) &&
($pass !== null && ($pass ==
Secret::read(Secret::getKey('imp'), $_SESSION['imp']['pass'])))) {
then FAILED
in the old code 4.O
if ((is_null($imapuser) || ($imapuser == $_SESSION['imp']['user'])) &&
(is_null($pass) || ($pass ==
Secret::read(Secret::getKey('imp'), $_SESSION['imp']['pass'])))) {
else FAILED
I don't understand.. I come with a CAS ticket using an horde auth,
then sometimes, the imp session is already created via the side bar
and you see a login failed in the imp framework..
Any ideas?. I'll patch to revert to get it working. I presume the else
has gone somewhere else..