pimutils / pimutils/vdirsyncer

read_only dav storage still creates a collection

Open
#857 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
Python
Stars
1.9k
Forks
185
Avg merge
11h 49m
Merged PRs (30d)
1

Description

I am playing around with vdirsyncer==0.16.8 and I intended to have set up my DAV backend as read_only, because I am not yet ready to let this program modify things on my server. So I configured read_only = true in my caldav storage. Because I couldn't make the integration with khal work (https://github.com/pimutils/khal/issues/991) I played around with the collections parameter which I've changed from null to ["collection"]. When running vdirsyncer discover it prompted me:

warning: No collection "tmms" found for storage baikal_tmms.
Should vdirsyncer attempt to create it? [y/N]: y

I accidentally hit "y". Mea culpa. But now I have a new collection on my read_only storage.
I expected the program to not modify my server, e.g. to error out when it encounters that the collection does not exist and the backend is configured as read-only. Bonuspoints if it shows me how print the existing collections and their capabilities to choose from.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the vdirsyncer discover entry point and trace how a missing collection is handled when the storage is configured with read_only = true. Confirm the behavior against a read-only DAV storage and ensure discovery does not create a collection; the existing-collections and capabilities display is an optional follow-up.

Written by the indexing model from the issue text.

Assessment

Tech stack
python
Domain
backend, cli
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.