Summary | Kronolith incorrectly refuses to update attendee upon response |
Queue | IMP |
Queue Version | Git master |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | jan (at) horde (dot) org |
Requester | bklang (at) horde (dot) org |
Created | 11/17/2011 (4981 days ago) |
Due | |
Updated | 11/24/2011 (4974 days ago) |
Assigned | 11/24/2011 (4974 days ago) |
Resolved | 11/24/2011 (4974 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
State ⇒ Resolved
updateAttendees() expects email addresses, not From: headers (
Bug #10772).3 files changed, 7 insertions(+), 2 deletions(-)
http://git.horde.org/horde-git/-/commit/2c9fb69c7ca152ea5c34b655e09cbf8c7873fdf5
Queue ⇒ IMP
State ⇒ Assigned
if (is_null($sender) || !preg_match('/<?$attendee>?/',
$sender_lcase)) {
Priority ⇒ 1. Low
Type ⇒ Bug
Summary ⇒ Kronolith incorrectly refuses to update attendee upon response
Queue ⇒ Kronolith
Milestone ⇒
Patch ⇒ No
State ⇒ Unconfirmed
Horde, replied using Kronolith. I received the reply via IMP and
attempted to update his status in the meeting. Kronolith refused,
saying "The attendee hasn't been updated because the update was not
sent from the attendee."
I tracked the cause down to a mismatch between the expected sender and
the observed sender. Because I had invited my friend using only his
email address his response was rejected. The problem line of code is
kronolith/lib/Api.php line 1388 which reads:
if (is_null($sender) || $sender_lcase == $attendee) {
In this case, $sender_lcase was "ben klang <bklang@horde.org>" and
$attendee was "bklang@horde.org"