vCard v2.1 parsing: BINARY value type recognized

[respiranto]

version: 4.5.5

When parsing v2.1 data, VALUE=BINARY is recognized if specified explicitly or implicitly if the property type is PHOTO or LOGO (see also #676). Most importantly, this means that data is base64-decoded / base64-encoded--regardless of whether ENCODING=BASE64 is also present.

vCard version 2.1 apparently does not have a value type BINARY. Futhermore, the ENCODING parameter is valid for all property types.

I would expect for v2.1 that

• a) VALUE=BINARY for v2.1 throws an exception or provides an error with validate() (see also vCard validation: Invalid value type should report an error #682),
• b) base64 decoding & encoding either
  • happens iff ENCODING=BASE64, regardless of value type, or
  • never happens.