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

LDP service: option to remove path/slug from resources #1101

Closed
srosset81 opened this issue Jan 20, 2023 · 1 comment
Closed

LDP service: option to remove path/slug from resources #1101

srosset81 opened this issue Jan 20, 2023 · 1 comment
Labels

Comments

@srosset81
Copy link
Contributor

srosset81 commented Jan 20, 2023

Problem
Having resources with URIs of type “/projects/semapps” is nice for the developer, but poses a number of problems:

It links the resource to an LDP container and prevents this container from being renamed/deleted.
It identifies the resource, whereas we'd like only those who can read it to know what it contains.

Proposed solution
Add an option to the LdpService so that all resources are created on the base URL (baseUrl) of the LDP server. When a resource is POSTed into a container, it is attached to the container. Nothing in the LDP spec prevents this from happening. The only requirement is that the URI of the newly created resource be returned.

This option would also ignore the “Slug” header.

To prevent the root container from having to display all LDP server resources, do not attach resources to the root container.

Translated with DeepL.com (free version)

@srosset81 srosset81 added the ldp label Jan 20, 2023
@srosset81
Copy link
Contributor Author

Done on #1151 with new resourcesWithContainerPath option

@srosset81 srosset81 changed the title LDP service: option pour ne plus mettre de path/slug aux ressources LDP service: option to remove path/slug from resources May 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant