Skip to content

Conversation

@rhafer
Copy link

@rhafer rhafer commented Sep 25, 2025

Related issue: opencloud-eu/opencloud#1458

For now this flag is used in this places:

  • the auth middlewares to reject tokens for users that do not have a tenant id assigned (excepte service users)
  • the ldap user manager refuses to start when not attribute mapping is defined for the tenantid
  • the ldap group manager refuse to start always

We also introduce a new group manager called "null" which is supposed to be used in multi-tenant setup and returns empty result or NOT FOUND for every request. (As simple way to address: opencloud-eu/opencloud#1407

Tested in opencloud PR: opencloud-eu/opencloud#1554

@rhafer rhafer self-assigned this Sep 25, 2025
For now this flag is used in the auth middlewares to reject tokens for
users that do not have a tenant id assigned.
Also the ldap user manager refuses to start when not attribute mapping
is defined for the tenantid

Related: opencloud-eu/opencloud#1458
Introduce the "null" group manager. It returns empty results for every
call. Supposed to be used in setups, where no groups are supported.

Also error out when the LDAP group manager is used when multi-tenancy
is enabled.
Copy link

@micbar micbar left a comment

Choose a reason for hiding this comment

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

Nice idea with the Null group manager.

@rhafer rhafer merged commit dcce351 into opencloud-eu:main Oct 2, 2025
19 checks passed
@openclouders openclouders mentioned this pull request Oct 2, 2025
1 task
rhafer added a commit to rhafer/opencloud that referenced this pull request Oct 2, 2025
rhafer added a commit to opencloud-eu/opencloud that referenced this pull request Oct 6, 2025
MahdiBaghbani pushed a commit to MahdiBaghbani/reva-opencloud that referenced this pull request Oct 17, 2025
…ncloud-eu#360)

Bumps [github.com/ceph/go-ceph](https://github.com/ceph/go-ceph) from 0.34.0 to 0.35.0.
- [Release notes](https://github.com/ceph/go-ceph/releases)
- [Changelog](https://github.com/ceph/go-ceph/blob/master/docs/release-process.md)
- [Commits](ceph/go-ceph@v0.34.0...v0.35.0)

---
updated-dependencies:
- dependency-name: github.com/ceph/go-ceph
  dependency-version: 0.35.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants