-
Notifications
You must be signed in to change notification settings - Fork 18
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
feat: multi tenancy #433
feat: multi tenancy #433
Conversation
…terRoomHash generator
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #433 +/- ##
==========================================
+ Coverage 40.57% 41.43% +0.85%
==========================================
Files 161 165 +4
Lines 17583 18151 +568
==========================================
+ Hits 7135 7521 +386
- Misses 10448 10630 +182 ☔ View full report in Codecov by Sentry. |
We should have custom config for each apps:
The console also need to show list off apps in each gateways, connectors |
* WIP: works app_sync logic, cluster with multi-tennancy by custom ClusterRoomHash generator * fix clippy * WIP: record with app * fix clippy warns * WIP: check app is same or not when re-join * handle app record, more app and record info to console APIs * update frontend * fixed: wrong migration script cause postgresql query error * fix: rtpengine create_answer error with sdp without connection line * fix sendtry init * feat: app with hook * fix warn
Description
This PR introduce multi tenancy feature. The original idea is from token generate step, instead of provide configured secret, we can have a list of apps with separated secret which can be pulled from other sources (like control panel).
I created a AppContext for each session for providing more custom behavior in future (now it only have app_id).
Logic:
Depend on node-types, we have difference mechanism:
Related Issue
If this pull request is related to any issue, please mention it here.
Checklist
Screenshots
If applicable, add screenshots to help explain the changes made.
Additional Notes
Add any additional notes or context about the pull request here.