[Tickets #14197] Re: Nag_Driver_Kolab stores parent_id strings base64 encoded (should be: unencoded)

[noreply at bugs.horde.org]

BITTE NICHT AUF DIESE NACHRICHT ANTWORTEN. NACHRICHTEN AN DIESE  
E-MAIL-ADRESSE WERDEN NICHT GELESEN.

Ticket-URL: https://bugs.horde.org/ticket/14197
------------------------------------------------------------------------------
  Ticket           | 14197
  Aktualisiert Von | Jan Schneider <jan at horde.org>
  Zusammenfassung  | Nag_Driver_Kolab stores parent_id strings base64 encoded
                   | (should be: unencoded)
  Warteschlange    | Kolab
  Typ              | Bug
  Status           | Resolved
  Priorität        | 1. Low
  Milestone        |
  Patch            |
  Zuständige       | Michael Rubinsky
------------------------------------------------------------------------------


Jan Schneider <jan at horde.org> (2015-12-29 17:35) hat geschrieben:

> Regarding this new detection logic in nag/lib/Driver/Kolab.php
>
>  +        /* Check if the decoded string resembles a uid hash...
> +         */
> +        if (!preg_match('/^[a-zA-Z0-9-]+$/', $decoded)) {
> +            return false;
> +        }
>
>
> The format of the Kolab UID's is not fixed and can contain ":" and
> other funky characters.
>
> I remember vaguely that's one of the reasons Jan changed the internal IDs
> when we reported that several backends were broken when using a Kolab driver
> + "special" UIDs that needed proper URI-encoding.
>
> @Jan: Do you remember anything about this?
>

I think there was some duplication of ID usage which was kind of  
normalized to UIDs across the code. It's well possible that  
accidentally the task parent IDs got somehow disconnected along the way.