hoodie / icalendar-rs

📆 icalendar library, in Rust of course - for fun
Apache License 2.0
122 stars 32 forks source link

Bad escape in TEXT properties #111

Closed good-praxis closed 1 week ago

good-praxis commented 1 week ago

As mentioned in #104, reviewing the spec I've found that colons do not get escaped in TEXT property fields

However, a COLON character in a "TEXT" property value SHALL NOT be escaped with a BACKSLASH character.

see here

The current implementation causes issues when importing into Proton Calendar, as unescaping colons isn't expected. I'll open a provisional PR and give that branch a test run, however I won't know if removing colon escaping there won't break it somewhere else