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

Implement Multi-Tenant DbContext for SQLite With Db Expiration #36

Merged
merged 12 commits into from
Oct 25, 2024

Conversation

deyvidnenchev
Copy link
Contributor

@deyvidnenchev deyvidnenchev commented Oct 9, 2024

If user of the Northwind API want to use his own dataset (not modified by other users) he has to include X-Tenant-ID header.

Tennant Based Northwind API drawio

  1. Add the ability to have fresh database based on X-Tenant-ID header of any request to Norhwind API
  2. Add DB expiration (SqlLite:memory)
  3. Add validation middleware for X-Tenant-ID header
  4. Remove InMemory Database option (useless because relations don't work)

@deyvidnenchev deyvidnenchev linked an issue Oct 9, 2024 that may be closed by this pull request
@pmoleri
Copy link
Contributor

pmoleri commented Oct 9, 2024

We should test if everything goes fine when multiple requests to the same tenant are done concurrently.
e.g. an App Builder view loading a category, products and orders grids at the same time.

Especially considering the seeding case, when there's not connection yet (or it has expired already).

@deyvidnenchev deyvidnenchev changed the title Implement multi-tenant db context for SQLite depending on the X-Tenan… Implement multi-tenant db context for SQLite With Db Expiration Oct 10, 2024
@deyvidnenchev deyvidnenchev changed the title Implement multi-tenant db context for SQLite With Db Expiration Implement Multi-Tenant db context for SQLite With Db Expiration Oct 10, 2024
@deyvidnenchev deyvidnenchev changed the title Implement Multi-Tenant db context for SQLite With Db Expiration Implement Multi-Tenant DbContext for SQLite With Db Expiration Oct 10, 2024
@deyvidnenchev deyvidnenchev self-assigned this Oct 11, 2024
@deyvidnenchev deyvidnenchev linked an issue Oct 15, 2024 that may be closed by this pull request
@deyvidnenchev deyvidnenchev removed their assignment Oct 16, 2024
@deyvidnenchev deyvidnenchev added the 💥 status: in-test PRs currently being tested label Oct 16, 2024
@hanastasov hanastasov removed the 💥 status: in-test PRs currently being tested label Oct 23, 2024
@pmoleri pmoleri merged commit c22519b into main Oct 25, 2024
3 checks passed
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.

Add Memory Monitoring Extent API with multi-tenant solution
4 participants