Summary | Missing auth credentials during ActiveSync prevent Horde from registering apps |
Queue | Synchronization |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | mrubinsk (at) horde (dot) org |
Requester | o+horde (at) immerda (dot) ch |
Created | 02/28/2013 (4550 days ago) |
Due | |
Updated | 03/06/2013 (4544 days ago) |
Assigned | 03/01/2013 (4549 days ago) |
Resolved | 03/06/2013 (4544 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
State ⇒ Resolved
commit 9a0be186efebbbef09c55b1cc3b39a8f7b227374
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Wed Mar 6 09:59:36 2013 -0500
Ensure we still have a Horde_Session::BEGIN value after destroy()
is called.
Final bit of
Bug: 12078framework/Core/lib/Horde/Session/Null.php | 11 +++++++++--
1 files changed, 9 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/9a0be186efebbbef09c55b1cc3b39a8f7b227374
commit 698a6fb5d0e88772a0cffa830ca1f62177b3c81a
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Wed Mar 6 09:58:20 2013 -0500
If we are going to extend Horde_Session, we need to allow access
to these properties.
Partial fix for
Bug: 12078framework/Core/lib/Horde/Session.php | 10 +++++-----
1 files changed, 5 insertions(+), 5 deletions(-)
http://git.horde.org/horde-git/-/commit/698a6fb5d0e88772a0cffa830ca1f62177b3c81a
The problem is, that in __get the begin property is proxied and
returned 0 if _active is not true. However, as _active is not exposed
to the subclass, the _active of the superclass won't be initialized,
hence in _get it's always false/null -> begin is always 0.
There is a different pull request that addresses this problem and
works on our side, like the old one already has. Could you please
review it and merge it? If you have a different proposal, we would be
happy if you can verify that your changes actually fix the problem.
Thanks.
commit 4da94bb496934fd303eca92876ccf9c7863accc4
Author: Michael J Rubinsky <mrubinsk@horde.org>
Date: Tue Mar 5 17:54:21 2013 -0500
Ensure Null sessions pass session length checks.
Possible fix for
Bug: 12078framework/Core/lib/Horde/Session/Null.php | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
http://git.horde.org/horde-git/-/commit/4da94bb496934fd303eca92876ccf9c7863accc4
I am unable to reproduce.
https://github.com/horde/horde/pull/32
works for me.
horde webmail edition 5.0.4 with Horde_ActiveSync-2.3.1 (installed via
pear on RHEL 6.4)
Priority ⇒ 1. Low
PHP being able to open the temporary session that the Null session
driver requires. Otherwise, someone who is experiencing this will
have to debug since I am unable to reproduce.
installed with pear.
State ⇒ Feedback
Queue ⇒ Synchronization
Patch ⇒ No
State ⇒ Unconfirmed
Milestone ⇒
Queue ⇒ Horde Framework Packages
Summary ⇒ Missing auth credentials during ActiveSync prevent Horde from registering apps
Type ⇒ Bug
Priority ⇒ 2. Medium
soon as something is to be synchronized, i get:
ERR: User is not authorized
I tracked this down to $session->exists('horde', 'auth/authId') not
being set in Registry::getAuth().
It might be that this only happens in combination with imap login and
it might be related to
Bug #11933.As a workaround i commented out in rpc.php:
// $session_control = 'none';
Now a session is properly set up on ActiveSync logins and thus horde
manages to register Apps for synchronization since getAuth() returns
the correct userid.