Add multitenancy support

[elhe26]

Hi,

As the title implies, adding multitenancy support would be an excellent addition to the extension.

Cloud functions have access to the admin sdk, so this won't cause any complex implementation.

• The collection could have a tenantId variable as optional (leave it empty).

The actual implementation is as follows:

...

// Tenant Instance
let auth: TenantAwareAuth = admin.auth().tenantManager().authForTenant(tenantId);

// User Evaluation
let userRecord: UserRecord = await auth.getUser(user_id);

...

Caveats:

• tenantId needs to be created (use admin.auth().tenantManager().createTenant)
• User needs to be added to tenant (use admin.auth().tenantManager().authForTenant(tenantId).createUser)

Regards,

Edit:

Created issue as per github comment , suggested by @philnash

[philnash]

Thanks for the clarification and example implementation here. That does seem relatively straightforward (though I'm sure the time it takes me to setup a multi-tenant application so I can test this will be the longest part 😄).

[elhe26]

Hopefully, you'll type fast, hehe.

[elhe26]

Hi @philnash, any updates?

[philnash]

I have returned to this repo for some work, but I haven't had the chance to check in on multi-tenancy stuff just yet.

I will try to put aside some time next week to take a look. Thanks for your patience @elhe26.

[elhe26]

Thanks, @philnash.

[elhe26]

Hey @philnash, any updates?

[elhe26]

Hey, @philnash, Following up on this. Any updates?

[philnash]

Hey @elhe26, I'm afraid that not long after you asked last time I was laid off from Twilio. So I'm no longer working on this extension. I see from the latest commits that @druv5319 has started working on the extension, perhaps they can weigh in?

[elhe26]

Hey @elhe26, I'm afraid that not long after you asked last time I was laid off from Twilio. So I'm no longer working on this extension. I see from the latest commits that @druv5319 has started working on the extension, perhaps they can weigh in?

Sorry to hear that @philnash :(

I hope you're ok.

[philnash]

I was very disappointed at the time. But I’m doing well now with a new job and new things to work on. Thanks @elhe26 🙂

[druv5319]

Hey @elhe26, I'm not actively adding any updates to the repo but I'll definitely take a look into adding multitenancy support.

[elhe26]

Hey @elhe26, I'm not actively adding any updates to the repo but I'll definitely take a look into adding multitenancy support.

Hi @druv5319 , I could create a pr for this since the change is just to expose variables.