Summary | Saving of Kolab XML objects with non-7bit elements fails |
Queue | Kolab |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | mrubinsk (at) horde (dot) org, thomas.jarosch (at) intra2net (dot) com |
Requester | mike.gabriel (at) das-netzwerkteam (dot) de |
Created | 12/16/2015 (3529 days ago) |
Due | |
Updated | 01/02/2016 (3512 days ago) |
Assigned | 12/29/2015 (3516 days ago) |
Resolved | 01/02/2016 (3512 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | Yes |
State ⇒ Resolved
New release dropped, closing.
Sounds like it's about time the new package gets dropped. Do you have
any outstanding issues that need to be addressed before this happens?
Sounds like it's about time the new package gets dropped. Do you have
any outstanding issues that need to be addressed before this happens?
https://github.com/horde/horde/commit/cc36df08d3aef06b72b00831ed82384f7e38461a
reproduce it with the current stable PEAR package so this has already
been fixed in Git. I'm trying to track down the exact commit(s) that
fix the issue, but it's definitely no longer happening.
thanks for going through all my recent bug reports. I highly appreciate that.
with non 7-bit characters?
this issue with all my previous posts on this bug report. Sorry for
that.
In the course of things, my first observation was:
o disable Horde caching
o Create an event (or task, or memo, ...) without umlauts
o Save the event (-> ok) (here as quoted-printable attachment)
o Edit the event, add an umlaut into the title or location or whereever.
o Save the event (-> event vanishes, at the latest when logging out
and in again)
Alternatively:
o disable Horde caching
o Create an event, add an umlaut into the title or location or whereever.
o Save the event (-> event vanishes, at the latest when logging out
and in again)
The above got fixed by the patch provided in
#14199. With#14199fixed, I now can see Kolab groupware objects containing Umlauts.
Things looked normal again with Horde caching enabled and disabled.
THEN... (coming to this bug)...
The next observation was:
o Edit an event already containing umlauts (that is now visible,
after
#14199has been fixed).Double check that this event is stored as base64 transfer-encoded
Kolab XML attachment.
o It doesn't matter if you add another umlaut or remove umlauts or whatever.
o Save the event (or task, or note, ...) -> results in the below
browser notification:
"""
Failed writing the Kolab object: Failed parsing Kolab object input
data of type string! Input was: Æioz»"¢}tzw(v)àQ1|z÷§¶÷«²*'×K¢wºå?...
[shortened to 50 characters]
"""
when saving/editing/syncing data containing 8bit data (and thus
base64 encoded in the imap message).
some gets raised by processing the "previous" object (which is not
really handled in the Kolab_Storage code, at all).
The issue got also observed by Jens and has been reported as
#13865.Jens tested my proposed (work-around) patch and reported success, as
well. See communication history of
#13865.Hope this extra information helps,
Mike
with non 7-bit characters?
With the "double decoding" patch applied, I don't have any issues when
saving/editing/syncing data containing 8bit data (and thus base64
encoded in the imap message).
Taken from Michael J Rubinsky <mrubinsk@horde.org>
Assigned to Michael J Rubinsky <mrubinsk@horde.org>
State ⇒ Assigned
going on.
@Michael: if "prio low" means this isn't included in a short-term
update of the PEAR packages, I totally disagree: Horde war
absolutely broken, modifying any appointment etc. was impossible for
us without this patch and it of course affected synching back items
from mobiles etc. THIS IS A VERY IMPORTANT FIX.
to do with the version that any potential fix will appear in. It's
just not high on any developer's personal @todo list. If this, or any
other issue, is critical to *you*, consider sponsoring development
time to work it.
@Michael: if "prio low" means this isn't included in a short-term
update of the PEAR packages, I totally disagree: Horde war absolutely
broken, modifying any appointment etc. was impossible for us without
this patch and it of course affected synching back items from mobiles
etc. THIS IS A VERY IMPORTANT FIX.
@Mike: Good work, thank you!
Priority ⇒ 3. High
Type ⇒ Bug
Summary ⇒ Saving of Kolab XML objects with non-7bit elements fails
Queue ⇒ Kolab
Milestone ⇒
Patch ⇒ Yes
New Attachment: 1003_Kolab_no-previous-object-storing.patch
State ⇒ Unconfirmed
characters.
A work-around patch is provided with this bug report, but the solution
probably needs a second look.
For details, see description in patch header.