Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Contacts with umlauts ä / ü / ö etc. are not imported #7026

Closed
0110101 opened this issue Sep 28, 2020 · 9 comments · Fixed by #7140
Closed

Contacts with umlauts ä / ü / ö etc. are not imported #7026

0110101 opened this issue Sep 28, 2020 · 9 comments · Fixed by #7140

Comments

@0110101
Copy link

0110101 commented Sep 28, 2020

Steps to reproduce

https://help.nextcloud.com/t/kontakte-android-synchronisieren-mit-davx5-in-nur-eine-richtung-one-way-sync/90544/17?u=faulpelz

Environment data

Android version: 9

Device model: Samsung Galaxy S8+

Nextcloud app version: 3.13.1

Nextcloud server version: 19.0.3

Logs

Web server error log


webdav | Sabre\DAV\Exception\UnsupportedMediaType: Validation error in vCard: ENCODING=QUOTED-PRINTABLE is not valid for this document type./var/www/nextcloud/3rdparty/sabre/dav/lib/CardDAV/Plugin.php - line 303:Sabre\CardDAV\Plugin->validateVCard()/var/www/nextcloud/3rdparty/sabre/event/lib/WildcardEmitterTrait.php - line 89:Sabre\CardDAV\Plugin->beforeCreateFile()/var/www/nextcloud/3rdparty/sabre/dav/lib/DAV/Server.php - line 1100:Sabre\DAV\Server->emit()/var/www/nextcloud/3rdparty/sabre/dav/lib/DAV/CorePlugin.php - line 527:Sabre\DAV\Server->createFile()/var/www/nextcloud/3rdparty/sabre/event/lib/WildcardEmitterTrait.php - line 89:Sabre\DAV\CorePlugin->httpPut()/var/www/nextcloud/3rdparty/sabre/dav/lib/DAV/Server.php - line 474:Sabre\DAV\Server->emit()/var/www/nextcloud/3rdparty/sabre/dav/lib/DAV/Server.php - line 251:Sabre\DAV\Server->invokeMethod()/var/www/nextcloud/3rdparty/sabre/dav/lib/DAV/Server.php - line 319:Sabre\DAV\Server->start()/var/www/nextcloud/apps/dav/lib/Server.php - line 320:Sabre\DAV\Server->exec()/var/www/nextcloud/apps/dav/appinfo/v2/remote.php - line 35:OCA\DAV\Server->exec()/var/www/nextcloud/remote.php - line 167:require_once("/var/www/ne ... p")
-- | --
@0110101 0110101 added the bug label Sep 28, 2020
@AndyScherzinger
Copy link
Member

@tobiasKaminsky for a detailed analysis please see the referenced forum post

@j-ed
Copy link

j-ed commented Sep 28, 2020

@tobiasKaminsky I think the given subject reflects only the result of the problem but not the root cause of it. It seems that the contact backup function creates vcf files in vCARD 3.0 format, which still contain vCARD 2.1-specific quoted-printable encoded entries. I've summarized my findings here.

@AndyScherzinger
Copy link
Member

Well it is even more complicated ;)
We get the export from Android (v2.1= and let the vCard lib convert it ti 3 for importability in Nextcloud) unfortunately the lib doesn't seem to take care of converting the quoted-printable encoded entities :(

maybe this could be achieved by using Apache Commons Codec

@tobiasKaminsky
Copy link
Member

lib doesn't seem to take care of converting the quoted-printable

Then please report this upstream.

@AndyScherzinger
Copy link
Member

Then please report this upstream.

@tobiasKaminsky Unfortunately I haven't found the time yet to report this to the lib project at https://github.com/mangstadt/ez-vcard/issues

@j-ed
Copy link

j-ed commented Oct 21, 2020

@AndyScherzinger , @tobiasKaminsky I've created the following issue ticket in the given repository for you:

mangstadt/ez-vcard#112

@AndyScherzinger
Copy link
Member

Thanks a lot @j-ed and it has been fixed by the maintainer already, so hoping for a quick release :)

@0110101
Copy link
Author

0110101 commented Oct 21, 2020

@AndyScherzinger .... what I am writing now is not meant seriously ;-)

Thanks to me ;-)

I spent hours testing it and posting it on the Nextcloud forum just wanted to mention it ;-)

Thanks to the supervisor and for the info.

@AndyScherzinger
Copy link
Member

@0110101 I am just happy if it solves the issue (not yet released or tested with Nextcloud...) and yes, you did a magnificent job analyzing the matter and finding the root cause 👏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants