3:58 a.m.
https://bugs.freedesktop.org/show_bug.cgi?id=56436
Priority: medium
Bug ID: 56436
CC: syncevolution-issues(a)syncevolution.org
Assignee: patrick.ohly(a)intel.com
Blocks: 55916
Summary: PIM API: avoid colon in peer UID
Severity: normal
Classification: Unclassified
OS: All
Reporter: patrick.ohly(a)gmx.de
Hardware: Other
Status: ASSIGNED
Version: unspecified
Component: SyncEvolution
Product: SyncEvolution
The API allowed colons as part of the peer UID. This special character is
avoided in SyncEvolution config file names and thus currently gets replaced by
underscores (SetPeer works, GetAllPeers returns different uid).
Instead of trying to figure out how to support a colon in UID, better remove it
from the API definition.
--
You are receiving this mail because:
You are on the CC list for the bug.
4:56 a.m.
New subject: [Bug 56436] PIM API: valid UID characters
https://bugs.freedesktop.org/show_bug.cgi?id=56436
Patrick Ohly <patrick.ohly(a)gmx.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|PIM API: avoid colon in |PIM API: valid UID
|peer UID |characters
--- Comment #1 from Patrick Ohly <patrick.ohly(a)gmx.de> ---
Uppercase characters are also problematic: GetAllPeers() returns the lower
case, while the database UID in EDS is in upper case.
Only allow lower case?
The alternative is to use the peerName property to store the original UID.
Makes the implementation more complicated, though.
--
You are receiving this mail because:
You are on the CC list for the bug.
1:25 a.m.
New subject: [Bug 56436] PIM API: valid UID characters
https://bugs.freedesktop.org/show_bug.cgi?id=56436
Patrick Ohly <patrick.ohly(a)gmx.de> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #2 from Patrick Ohly <patrick.ohly(a)gmx.de> ---
(In reply to comment #0)
The API allowed colons as part of the peer UID. This special
character is
avoided in SyncEvolution config file names and thus currently gets replaced
by underscores (SetPeer works, GetAllPeers returns different uid).
Instead of trying to figure out how to support a colon in UID, better remove
it from the API definition.
Done.
(In reply to comment #1)
Uppercase characters are also problematic: GetAllPeers() returns the
lower
case, while the database UID in EDS is in upper case.
Only allow lower case?
Actually, that was the orginal proposal. The implementation just didn't check
it. Added such a check.
--
You are receiving this mail because:
You are on the CC list for the bug.
3579
days inactive
3614
days old
syncevolution-issues@syncevolution.org
2 comments
1 participants
participants (1)
-
bugzilla-daemon＠freedesktop.org