fix: accept quoted parameters

This are necessay when escaping semicolons, colons or commas.

src/parser/components.rs

@@ -456,7 +456,7 @@ fn test_multi_properties() {
                 params: vec![
                     Parameter {
                         key: "EMAIL".into(),
-                        val: Some("\"[email protected]\"".into()),
+                        val: Some("[email protected]".into()),
                     },
                     Parameter {
                         key: "CUTYPE".into(),
@@ -471,7 +471,7 @@ fn test_multi_properties() {
                 params: vec![
                     Parameter {
                         key: "EMAIL".into(),
-                        val: Some("\"[email protected]\"".into()),
+                        val: Some("[email protected]".into()),
                     },
                     Parameter {
                         key: "CUTYPE".into(),

src/parser/parameters.rs

@@ -1,11 +1,11 @@
 use nom::{
     branch::alt,
-    bytes::complete::{tag, take_till1},
+    bytes::complete::{is_not, tag, take_till1},
     character::complete::space0,
     combinator::{eof, opt},
     error::{convert_error, ContextError, ParseError, VerboseError},
     multi::many0,
-    sequence::{preceded, separated_pair, tuple},
+    sequence::{delimited, preceded, separated_pair, tuple},
     Finish, IResult, Parser,
 };
 
@@ -141,7 +141,12 @@ fn pair_parameter<'a, E: ParseError<&'a str> + ContextError<&'a str>>(
         separated_pair(
             valid_key_sequence_cow, //key
             tag("="),
-            opt(alt((eof, take_till1(|x| x == ';' || x == ':')))).map(remove_empty_string),
+            opt(alt((
+                eof,
+                delimited(tag("\""), is_not("\""), tag("\"")),
+                take_till1(|x| x == ';' || x == ':'),
+            )))
+            .map(remove_empty_string),
         ),
     )
     .map(|(key, val)| Parameter {
@@ -161,7 +166,12 @@ fn base_parameter<'a, E: ParseError<&'a str> + ContextError<&'a str>>(
         ),
         opt(preceded(
             tag("="),
-            alt((eof, take_till1(|x| x == ';' || x == ':'))).map(ParseString::from),
+            alt((
+                eof,
+                delimited(tag("\""), is_not("\""), tag("\"")),
+                take_till1(|x| x == ';' || x == ':'),
+            ))
+            .map(ParseString::from),
         ))
         .map(remove_empty_string_parsed),
     ))
@@ -195,6 +205,11 @@ pub fn parse_parameter_list() {
             Parameter::new_ref("DATE", Some("20170218")),
         ]
     );
+    assert_parser!(
+        parameters,
+        ";TEXT=\"quoted text with \\;\"",
+        vec![Parameter::new_ref("TEXT", Some("quoted text with \\;")),]
+    );
 }
 
 pub fn parameters<'a, E: ParseError<&'a str> + ContextError<&'a str>>(