Skip to content
This repository was archived by the owner on Apr 13, 2022. It is now read-only.
This repository was archived by the owner on Apr 13, 2022. It is now read-only.

proposal: deleting a container is a noop #173

Description

@michielbdejong

The spec already says that creating a container is automatic (mkdir -p).

So if you create and then delete /foo/bar.ttl, it seems asymmetric that you then do have to delete the /foo/ container afterwards if you no longer want to see foo as a member listed in /.

This would also resolve confusion around how exactly container deletes should work (should sub-trees also be deleted, should they fail if the container is non-empty, what should happen if you deleted a resource but not its ACL, how can you know if any ACL docs exists, if they are not listed as members, etc.)

Activity

  1. jeff-zucker commented on May 22, 2019

    @jeff-zucker
    Member

    My reading of the spec is that creating a container (via POST) is manual. That creating a container on its own with PUT is not allowed. That a container may be created as a by-product of creating resources in it. So deleting a container is symmetric with creating one with POST and should be manual. There are also use cases in which one wants to delete all the resources in a container but not delete the container.

  2. akuckartz commented on May 22, 2019

    @akuckartz

    There are also use cases in which one wants to delete all the resources in a container but not delete the container.

    Which use cases are these?

  3. jeff-zucker commented on May 22, 2019

    @jeff-zucker
    Member

    Re-Initializing a folder tree for an app. But because of the way PUT operates, deleting it or not deleting it is pretty much the same so nevermind.

  4. jeff-zucker commented on May 22, 2019

    @jeff-zucker
    Member

    And what about creating/foo/bar/baz.ttl. When we delete we might want to delete baz.ttl And /foo/bar without wanting to delete /foo But if it's symmetric with the create it would be.

  5. jeff-zucker commented on May 22, 2019

    @jeff-zucker
    Member

    Ah, here's a use case: I create /foo/jamal/ to hold messages from Jamal and I create a .acl file to give Jamal write access to that folder. Just because I delete the last message from Jamal doesn't mean I want to delete the folder and its .acl.

  6. michielbdejong commented on May 23, 2019

    @michielbdejong
    ContributorAuthor

    a container with a .acl document in there is not an empty container.

    I mainly proposed this to track it as an option, I think the current behaviour is fine as it is, and don't think it's worth a spec change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions