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.

Sign up for GitHub

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

Jump to bottom

[BUG] Fix configs to support multi-tenancy #346

Open
roberlander2 opened this issue Jan 26, 2023 · 0 comments · May be fixed by #359
Open

[BUG] Fix configs to support multi-tenancy #346

roberlander2 opened this issue Jan 26, 2023 · 0 comments · May be fixed by #359
Assignees
@roberlander2
Labels
bug Something isn't working

Comments

@roberlander2
Copy link
Collaborator

Describe the bug
The current implementation of ApplicationConfig does not support multi-tenancy. Only one application config is used (based on environment variables), so any functions using data from an ApplicationConfig will not support multi-tenancy.

Expected behavior
Implement the configs collection solution from https://github.com/rokwire/building-block-template-go and add Type, AppID, and OrgID fields to support multi-tenancy. Also, a System flag should be added to configs to allow system-level users to control which configs may be modified by lower-level admins. SyncConfig and ManagedGroupConfig should both be stored in the configs collection.
@roberlander2 roberlander2 added the bug Something isn't working label Jan 26, 2023
@roberlander2 roberlander2 self-assigned this Jan 26, 2023
@roberlander2 roberlander2 mentioned this issue Jan 26, 2023
Open
roberlander2 added a commit that referenced this issue Jan 26, 2023
@roberlander2
start updating configs to match bb template [#346]
5c4ee7c
@shurwit shurwit mentioned this issue Jan 30, 2023
Open
18 tasks
roberlander2 added a commit that referenced this issue Feb 13, 2023
@roberlander2
continue to add configs and fix resulting errors [#346]
544982c
roberlander2 added a commit that referenced this issue Feb 14, 2023
@roberlander2
more config updates, use new tokenauth.Claims.CanAccess function [#346]
24fa2df
roberlander2 added a commit that referenced this issue Feb 14, 2023
@roberlander2
more multi-tenancy support progress [#346]
b6f903e
roberlander2 added a commit that referenced this issue Feb 15, 2023
@roberlander2
update APIs and internal functions to not rely on clientID from heade…
7b20cf0 
…r, fix go mod [#346]
roberlander2 added a commit that referenced this issue Feb 16, 2023
@roberlander2
fix more errors from removing clientID [#346]
b008159
roberlander2 added a commit that referenced this issue Feb 16, 2023
@roberlander2
finish fixing build errors, upgrade groups bb to go1.20.1 [#346]
b7948b8
roberlander2 added a commit that referenced this issue Feb 18, 2023
@roberlander2
update database init for multi-tenancy, configs migrations [#346]
c07418f
@roberlander2 roberlander2 linked a pull request Feb 21, 2023 that will close this issue
Open
18 tasks
@roberlander2 roberlander2 linked a pull request Feb 21, 2023 that will close this issue
[#346] Fix configs to support multi-tenancy #359
Open
18 tasks
roberlander2 added a commit that referenced this issue Feb 23, 2023
@roberlander2
handle multiple managed groups per appOrg [#346]
be4816f
roberlander2 added a commit that referenced this issue Mar 16, 2023
@roberlander2
upgrade to auth-lib v3 [#346]
ce118f9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@roberlander2 roberlander2

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#346] Fix configs to support multi-tenancy rokwire/groups-building-block
1 participant
@roberlander2