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

Specify that DFC LDP servers must support PATCH and/or PUT #7

Open
lecoqlibre opened this issue Nov 24, 2023 · 4 comments
Open

Specify that DFC LDP servers must support PATCH and/or PUT #7

lecoqlibre opened this issue Nov 24, 2023 · 4 comments
Assignees

Comments

@lecoqlibre
Copy link
Member

@simonLouvet We should specify that DFC LDP servers must support PATCH and/or PUT as the LDP spec does not require servers to support them:

4.2.4 HTTP PUT

Per [RFC7231], this HTTP method is optional and this specification does not require LDP > servers to support it.

4.2.7 HTTP PATCH

Per [RFC5789], this HTTP method is optional and this specification does not require LDP >servers to support it.

@simonLouvet simonLouvet self-assigned this Dec 6, 2023
@simonLouvet
Copy link
Contributor

exact. Put used because patch involves a great deal of complexity

@RachL
Copy link
Member

RachL commented Apr 22, 2024

@simonLouvet @lecoqlibre can we close this one?

@simonLouvet
Copy link
Contributor

simonLouvet commented Apr 23, 2024

@RachL No, question of @lecoqlibre don't have explicit answer.

I think DFC servers have to implement PUT because it's the easiest way to update data on one platform from another. On the other hand, we can rule out PATCH, which is complex to specify. Is ok for you @lecoqlibre ?

@lecoqlibre
Copy link
Member Author

lecoqlibre commented Apr 24, 2024

I would like the standard to have a possibility to patch data. PATCH is very useful as it allows to change only a part of the data. Also it consumes less resources as we only pass the modified data and not the whole dataset.

I was thinking about using N3 patch like in the Solid protocol.

Why do you think it's complex to specify @simonLouvet?

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

No branches or pull requests

3 participants