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

Allow template option Configuration.MultiTenancy.IsEnabled = true; Global DataFilter for Tenant entities #27006

Closed
papyr opened this issue Oct 17, 2020 · 7 comments
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates enhancement This issue represents an ask for new feature or an enhancement to an existing one multi-tenancy Issues based in multi-tenancy
Milestone

Comments

@papyr
Copy link

papyr commented Oct 17, 2020

Please allow Configuration.MultiTenancy.IsEnabled = true; Global DataFilter for Tenant entities enabled/Disabled options.

ASP templates should be configurable to allow Global Data Filters projected on tenant specific data.

There are many options as listed below, the default should allow something easy and quick for the developer to switch on.

  1. Using database per tenant strategy
  2. Using shared database for tenants
  3. Multi-tenant EF Core database context with security net
  4. Loading tenants configuration
  5. Missing tenant middleware
  6. Tenants provider to get current tenant
  7. Tenant-based dependency injection
  8. Unit-testing multi-tenant database context (coming soon)
  9. Multi-tenant configurable composite commands

There are many lib options often don't get maintained, some get left out or fall off

But this is a good example

@pranavkm pranavkm added the area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates label Oct 17, 2020
@mkArtakMSFT mkArtakMSFT added the enhancement This issue represents an ask for new feature or an enhancement to an existing one label Oct 19, 2020
@mkArtakMSFT mkArtakMSFT added this to the Backlog milestone Oct 19, 2020
@ghost
Copy link

ghost commented Oct 19, 2020

We've moved this issue to the Backlog milestone. This means that it is not going to be worked on for the coming release. We will reassess the backlog following the current release and consider this item at that time. To learn more about our issue management process and to have better expectation regarding different types of issues you can read our Triage Process.

@blowdart blowdart added the multi-tenancy Issues based in multi-tenancy label Oct 19, 2020
@javiercn
Copy link
Member

javiercn commented Nov 6, 2020

@papyr thanks for contacting us.

We recommend using something like Orchard Core to support these types of scenarios. We are not clear what the ask is here, but we don't plan to ship a template with multitenancy out of the box since there are other libraries that offer this capability.

However, if there are specific issues that block you from implementing multitenancy within your app, we would evaluate them and consider possible solutions.

@javiercn javiercn closed this as completed Nov 6, 2020
@papyr
Copy link
Author

papyr commented Nov 7, 2020

Sorry, you're working for Microsoft yet you dont know/how can you not be clear on what supporting Multi-Tenancy is???

Look at the number of up votes from your developer community here who all seem to get it... they all seem to get it and are asking for it!

image

@davidfowl
Copy link
Member

We build an extensible platform so we don't always need to provide all of the pieces to the puzzle. We also need to make sure feature like this one in particular can work without making the design of what we have inefficient (that's one of the main concerns I have). As long as it's possible to build a multi-tenant solution on the foundation easily we as a team don't need to build in support for it and those many components required in specific kinds of applications.

I think multi tenancy and SASS is important and needs to be possible to do with our components.

@papyr
Copy link
Author

papyr commented Nov 8, 2020

Tenancy sits on the platform infrastructure, including hosting, gateways, portals and more!.

Its clear you many in the MS world are unaware and have not read those standards, or are disconnected from the developers need. So, IMHO its not an opinion for you to be heavy handedly closing this,

its an industry standard in pretty much all the stacks, there are so many examples of well organized and prescribed

approaches on these by the J2EE community, Apple, Visual Force, PHP and many others, since it also pertain to security! Leaving this open could also make MS vulnerable to the fact that they neglecting security on purpose after repeated attempts by the community asking them to fix this critical need. There are so so many developers who are asking for this before you closed the voting here and user voice!

image

To your point, people can extend or customize, like the razor engine etc. but a native implementation is required and missing in the stack, it hinders adoption for smaller organizations and startups.

@davidfowl
Copy link
Member

davidfowl commented Nov 8, 2020

I don't know what you mean by industry standard. There are many 3rd party plugins that add an opinionated way to do multi tenancy with ASP.NET Core to build SASS application (and there are people using ASP.NET Core to build multi tenant applications just fine)

This GitHub issue should be used to discuss concrete concerns and difficulties with various existing approaches and to suggest what we might do to ease that pain.

Gross generalizations and talk of "industry standards" are non constructive.

@papyr
Copy link
Author

papyr commented Nov 10, 2020

If you don't know what those standards are click on the link, there is not a gross generalization, these are committees and boards and publications. Are you really at Microsoft!!!

If multi-tenancy is not concrete then what is is, Fiction?? hahahahaaa

You must arrogant to ignore the highest one of the highest rate & requested feature on Microsoft own poll.

@ghost ghost locked as resolved and limited conversation to collaborators Dec 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates enhancement This issue represents an ask for new feature or an enhancement to an existing one multi-tenancy Issues based in multi-tenancy
Projects
None yet
Development

No branches or pull requests

6 participants