From 740d6706659a6780d426b22829c01451b2ca3f5c Mon Sep 17 00:00:00 2001 From: hackerESQ Date: Thu, 22 Apr 2021 04:09:16 -0500 Subject: [PATCH] Adds ability to disable tenant asset routes (#636) * Add option to enable/disable tenant asset route * Only registers tenant route if enabled in config * Uses proper config key * Move routes config option * Move config to service provider * Moves config to service provider --- assets/config.php | 9 +++++++++ src/TenancyServiceProvider.php | 4 +++- 2 files changed, 12 insertions(+), 1 deletion(-) diff --git a/assets/config.php b/assets/config.php index 21529ecb..d992420e 100644 --- a/assets/config.php +++ b/assets/config.php @@ -168,6 +168,15 @@ return [ // Stancl\Tenancy\Features\TenantConfig::class, // https://tenancyforlaravel.com/docs/v3/features/tenant-config // Stancl\Tenancy\Features\CrossDomainRedirect::class, // https://tenancyforlaravel.com/docs/v3/features/cross-domain-redirect ], + + /** + * Should tenancy routes be registered. + * + * Tenancy routes include tenant asset routes. By default, this route is + * enabled. But it may be useful to disable them if you use external + * storage (e.g. S3 / Dropbox) or have a custom asset controller. + */ + 'routes' => true, /** * Parameters used by the tenants:migrate command. diff --git a/src/TenancyServiceProvider.php b/src/TenancyServiceProvider.php index 946f114f..cf326792 100644 --- a/src/TenancyServiceProvider.php +++ b/src/TenancyServiceProvider.php @@ -110,7 +110,9 @@ class TenancyServiceProvider extends ServiceProvider __DIR__ . '/../assets/TenancyServiceProvider.stub.php' => app_path('Providers/TenancyServiceProvider.php'), ], 'providers'); - $this->loadRoutesFrom(__DIR__ . '/../assets/routes.php'); + if (config('tenancy.routes', true)) { + $this->loadRoutesFrom(__DIR__ . '/../assets/routes.php'); + } $this->app->singleton('globalUrl', function ($app) { if ($app->bound(FilesystemTenancyBootstrapper::class)) {