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

writemultiple has difficult semantics, how about read only? Best effort approach? Fail, if one of the writes is failing? #1643

Open
Tracked by #1613
mlagally opened this issue Aug 3, 2022 · 4 comments

Comments

@mlagally
Copy link
Contributor

mlagally commented Aug 3, 2022

No description provided.

@github-actions github-actions bot added the needs-triage Automatically added to new issues. TF should triage them with proper labels label Aug 3, 2022
@sebastiankb
Copy link
Contributor

I'm not sure if I understand this issue correctly. Please not that writemultipleproperties is already part of TD 1.0.

how about read only?

We have readmultipleproperties and readallproperties.

Best effort approach? Fail, if one of the writes is failing?

This is a question that depends on the protocol used and should be clarified in the protocol binding TF.

@mlagally
Copy link
Contributor Author

mlagally commented Aug 5, 2022

@sebastiankb
My comment was probably a bit brief. The question is how writemultipleproperties should behave, when somebody tries to write a set of properties of which some are read-only. Should the entire operation fail? Should it succeed without notifying the caller that he made a mistake? Is it implementation dependent?

@benfrancis
Copy link
Member

@sebastiankb

This is a question that depends on the protocol used and should be clarified in the protocol binding TF.

For example, this is being discussed for the HTTP Baseline Profile protocol binding in w3c/wot-profile#162

@sebastiankb
Copy link
Contributor

sebastiankb commented Aug 8, 2022

My comment was probably a bit brief. The question is how writemultipleproperties should behave, when somebody tries to write a set of properties of which some are read-only. Should the entire operation fail? Should it succeed without notifying the caller that he made a mistake? Is it implementation dependent?

This is a good point but this is also true for other operations. E.g., so far the TD have not predefined how the servient should behave when, e.g., a write call is executed on a property that can only be read. TD offers the opportunity to define multiple responses which can be used to express if writemultipleproperties was successful or partial successful or not successful with specific reasons. The Profile can take advantage of this and establish certain guidelines.

@sebastiankb sebastiankb removed the needs-triage Automatically added to new issues. TF should triage them with proper labels label Aug 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants