Skip to content

Conversation

@bevzzz
Copy link
Collaborator

@bevzzz bevzzz commented Apr 7, 2025

This PR brings support for dynamic user management features introduced in v1.30.

Adds (sync/async):

  • DbUsers and OidcUsers clients accessible via client.users().db() and client.users().oidc()
  • UserAssignemntsGetter to retrieve uses assigned to the role from /user-assignments endpoint
  • Another Result constructor that allows to override the body while reusing the HTTP response
  • Result.voidToBoolean helpers now accept a list of allowed failure codes that still should produce true in the result
  • array-to-list parsers for Result
  • Extends RoleAssigner and RoleRevoker to take an optional namespace argument. This allows re-using the existing classes for legacy, db-, and oidc- users.

Deprecates (sync/async):

  • assigner, revoker, and userRolesGetter on the root Users client (not namaspaced)
  • assignedUsersGetter in Roles client (in favor of userAssignmentsGetter)
  • Enables OIDC and dynamic user management in WeaviateWithRbac testcontainer

Updates:

  • Weaviate version and commit hash for integration tests
  • WCS_DUMMY email used for OIDC tests (see thread)

Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@bevzzz bevzzz merged commit 4df2216 into main Apr 9, 2025
5 checks passed
@bevzzz bevzzz deleted the feat/dynamic-user-managament branch April 9, 2025 13:26
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

Successfully merging this pull request may close these issues.

3 participants