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

PutUserToSharedFolder Method Fails to Update Existing Users' Permissions in Shared Folder - Error: (key type required) #135

Open
YoussefFenetre opened this issue Nov 22, 2024 · 2 comments

Comments

@YoussefFenetre
Copy link

I'm experiencing an issue with the Keeper SDK when trying to update the permissions of existing users in a shared folder using the PutUserToSharedFolder method. While adding new users works correctly, attempting to update existing users' permissions results in an error indicating that a key type is required.

Steps to Reproduce:

  1. Use the PutUserToSharedFolder method to add new users to a shared folder successfully.
  2. Attempt to update the permissions of an existing user in the shared folder using the same method.
  3. Observe that the method fails with an error message.

Error Message and Stack Trace:

Error adding users to shared folder: An error has occurred. Please check if an update is available for your device on the App Store or the Keeper Security website. If you continue to receive this error, please email [email protected] or contact us at https://keepersecurity.com/support.html and let us know that error code (key type required) has occurred.

Error adding users to shared folder: at KeeperSecurity.Authentication.KeeperEndpoint.ExecuteRest(String endpoint, ApiRequestPayload payload)
at KeeperSecurity.Authentication.AuthCommon.ExecuteAuthRest(String endpoint, IMessage request, Type responseType, Int32 apiVersion)
at KeeperSecurity.Authentication.AuthExtensions.ExecuteAuthRest[TC,TR](IAuthentication auth, String endpoint, TC request, Int32 apiVersion)
at KeeperSecurity.Vault.VaultOnline.PutUserToSharedFolder(String sharedFolderUid, String userId, UserType userType, ISharedFolderUserOptions options)
at MyAppNamespace.ViewModel.FolderManagerViewModel.AddUsersToSharedFolder(String sharedFolderUid)

Additional Information:

Behavior Observed:

  • Adding new users to the shared folder using PutUserToSharedFolder works without issues.
  • Updating permissions of existing users using the same method results in the error above.

Analysis:

  • It appears that when updating existing users, the PutUserToSharedFolder method does not handle the operation correctly and requires additional parameters or modifications.
  • Including the shared folder key in SharedFolderUserOptions is not feasible as per the method's intended usage and security considerations.

Request:

  • Guidance: If there's an alternative recommended approach for updating existing users' permissions, please provide guidance on how to implement it using the Keeper SDK.

Thank you for your assistance!

If any additional information is needed, please let me know.

@sk-keeper
Copy link
Collaborator

Are you trying to add yourself to a shared folder?

@sk-keeper
Copy link
Collaborator

This issue could be duplicated when a user adds/modifies its permissions in a shared folder.
Add permissions error originates at the .Net SDK. The fix can be release any time.
Update its own permissions is a backend issue. It is already fixed by the backend team.
It is hard to tell when the fix will be released.

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

2 participants