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

share_with_user_type is not set in response #3622

Open
phil-davis opened this issue Apr 29, 2022 · 4 comments
Open

share_with_user_type is not set in response #3622

phil-davis opened this issue Apr 29, 2022 · 4 comments
Labels

Comments

@phil-davis
Copy link
Contributor

phil-davis commented Apr 29, 2022

This was introduced in oC10 core owncloud/core#40013 and needs to be added to oCIS-reva

@micbar
Copy link
Contributor

micbar commented May 11, 2022

@phil-davis @kulmann What is the use case of that property?

@phil-davis
Copy link
Contributor Author

@phil-davis @kulmann What is the use case of that property?

It helps clients to be able to know if the share receiver (sharee) is a real user or a guest user. If "guest" users do not exist in oCIS it will make no difference, but it could be added and would always be the "user" value at the moment on oCIS.

Docs issue owncloud/docs-server#272
"we emit a new field called share_with_user_type which might either be 0 = user or 1 = guest"

So it could be set to "0".

Note: I raise these issues so that an issue exists in case some test scenario from oC10 core fails, that makes it easy for the difference to be linked to an issue in expected-failures, which can then be discussed (like is happening here)

@kulmann
Copy link
Member

kulmann commented May 12, 2022

There are two occurences of that field. One in ocs sharee query responses so that we know which type of user we're dealing with when inviting people. That's probably needed for ocis/reva as well if we don't come up with an entirely different guest sharing solution.

For the ocs share(s) endpoints it would be better if we emit the proper guest share type (4 instead of 0) for guest shares. The user type for shares is only used in the sdk (api abstraction layer) to rewrite shares with type 0 and user type 1 to guest shares (type 4). So it's a workaround in oc10, where we didn't want to take the chance of compromising other pieces of code that rely in a share type 0 for guest shares.

@kulmann
Copy link
Member

kulmann commented May 12, 2022

Wouldn't hurt to emit the user type of the share as well, but it's not needed for web so I would not wxpect it to be added as long as we get proper guest share types.

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

3 participants