Question about this package

[tomhatzer]

Hey there!

Thanks for creating this package and providing it as open source.

I've just got a small question about it: Is this package basically only for logged in/authorized users?

I'd love to use this to provide multitenancy for non-authorized users, basically a system where there are multiple websites running on different domains or subdomains and non-authorized users can visit those domains and browse through the contents of the corresponding tenant.

There's currently one part of code that's denying this:
https://github.com/romegadigital/Multitenancy/blob/98796699b2f4766510d00c63d614fc8b1ee622e3/src/Middleware/TenantMiddleware.php#L40-L42

This snippet is triggered every single time as my users are not logged in (just regular website visitors). Would it be possible to make this optional (through a variable in the config or .env file maybe?). I could also provide a PR for this if you want.

Thank you very much!

Best wishes
Tom

[bradenkeith]

This is coming very soon. I have a need for it in my current project as well. For now you can just copy the project’s middleware into your app and alter the login checks to match your scenario. Feel free to provide a PR if you’d like.

…

On Fri, Mar 22, 2019 at 19:07 Tom Hatzer ***@***.***> wrote: Hey there! Thanks for creating this package and providing it as open source. I've just got a small question about it: Is this package basically only for logged in/authorized users? I'd love to use this to provide multitenancy for non-authorized users, basically a system where there are multiple websites running on different domains or subdomains and non-authorized users can visit those domains and browse through the contents of the corresponding tenant. There's currently one part of code that's denying this: https://github.com/romegadigital/Multitenancy/blob/98796699b2f4766510d00c63d614fc8b1ee622e3/src/Middleware/TenantMiddleware.php#L40-L42 This snippet is triggered every single time as my users are not logged in (just regular website visitors). Would it be possible to make this optional (through a variable in the config or .env file maybe?). I could also provide a PR for this if you want. Thank you very much! Best wishes Tom — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#14>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AANYos_KkX9Rz7U0Pofz-VlkD4xSzN-sks5vZWGngaJpZM4cEjJc> .

[bradenkeith]

Resolved in #15