search

owncloud / contacts

:busts_in_silhouette: Manage your CardDAV contacts from a feature-rich web-interface
75 stars 46 forks source link

Invalid VCard exporting and via CardDAV #401

Open llucax opened 8 years ago

llucax commented 8 years ago

Coming the Android app DAVdroid: https://forums.bitfire.at/topic/1078/error-while-uploading-created-modified-entries/20

Here is what the author said:

According to RFC 6350, »property parameter values must not contain the DQUOTE (U+0022) character« (details in the RFC, see the definition of ADR LABEL as param-value, and param-value can't contain "). A possible solution could be to use RFC 6868.

Steps to reproduce

  1. Create a contact with a " character in the address Get the contact via CardDAV
  2. Get the contact via CardDAV or download feature

    Expected behaviour

A valid vCard is downloaded or received via CardDAV (either " is removed, changed to something else, or RFC 6868 is used)

Actual behaviour

The " appears in the vCard, making the vCard invalid.

Server configuration

Operating system: Ubuntu 15.10

Web server: Apache

Database: MySQL

PHP version: 5.6.11+dfsg-1ubuntu3.3

ownCloud version: 9.0.2-1.1

Updated from an older ownCloud or fresh install: Updated

Where did you install ownCloud from: Debian/Ubuntu repo

Signing status (ownCloud 9.0 and above): all passes

List of activated apps:

Enabled:
  - activity: 2.2.1
  - calendar: 1.1
  - comments: 0.2
  - contacts: 1.2.0.0
  - dav: 0.1.6
  - documents: 0.12.0
  - external: 1.2
  - federatedfilesharing: 0.1.0
  - federation: 0.0.4
  - files: 1.4.4
  - files_external: 0.5.2
  - files_pdfviewer: 0.8.1
  - files_sharing: 0.9.1
  - files_texteditor: 2.1
  - files_trashbin: 0.8.0
  - files_versions: 1.2.0
  - files_videoplayer: 0.9.8
  - firstrunwizard: 1.1
  - gallery: 14.5.0
  - mail: 0.4.2
  - notifications: 0.2.3
  - provisioning_api: 0.4.1
  - search_lucene: 0.6.0
  - sharelinks: 1.0.1
  - systemtags: 0.2
  - tasks: 0.9.0
  - templateeditor: 0.1
  - updatenotification: 0.1.0

The content of config/config.php:

Are you using external storage, if yes which one: no

Are you using encryption: no

Are you using an external user-backend, if yes which one: no

I cut down some fields from the template that seemed irrelevant, let me know if you need something else.

llucax commented 8 years ago

First reported to core: https://github.com/owncloud/core/issues/24448

javaboyuk commented 8 years ago

Do you think this will make it into 1.5 ?