diff --git a/app/Providers/SamlServiceProvider.php b/app/Providers/SamlServiceProvider.php index 18b93cf56..c9e50dafe 100644 --- a/app/Providers/SamlServiceProvider.php +++ b/app/Providers/SamlServiceProvider.php @@ -49,10 +49,6 @@ class SamlServiceProvider extends ServiceProvider 'uses' => 'Auth\SamlController@login', ] ); - Route::group(['prefix' => 'admin', 'middleware' => ['web', 'auth', 'authorize:superuser']], function () { - Route::get('saml', ['as' => 'settings.saml.index', 'uses' => 'SettingsController@getSamlSettings']); - Route::post('saml', ['as' => 'settings.saml.save', 'uses' => 'SettingsController@postSamlSettings']); - }); }); } diff --git a/routes/web.php b/routes/web.php index 68bd4f33f..b6f2655b4 100644 --- a/routes/web.php +++ b/routes/web.php @@ -287,6 +287,19 @@ Route::group(['prefix' => 'admin', 'middleware' => ['auth', 'authorize:superuser ->push(trans('admin/settings/general.login'), route('settings.logins.index'))); + // SAML + Route::get('/saml', [SettingsController::class, 'getSamlSettings']) + ->name('settings.saml.index') + ->breadcrumbs(fn (Trail $trail) => + $trail->parent('settings.index') + ->push(trans('admin/settings/general.saml_title'), route('settings.saml.index'))); + + Route::post('/saml', [SettingsController::class, 'postSamlSettings']) + ->name('settings.saml.save'); + + + + // Backups Route::group(['prefix' => 'backups', 'middleware' => 'auth'], function () { Route::get('download/{filename}', @@ -321,11 +334,16 @@ Route::group(['prefix' => 'admin', 'middleware' => ['auth', 'authorize:superuser Route::resource('groups', GroupsController::class); + + /** + * This breadcrumb is repeated for groups in the BreadcrumbServiceProvider, since groups uses resource routes + * and that servcie provider cannot see the breadcrumbs defined below + */ Route::get('/', [SettingsController::class, 'index']) ->name('settings.index') ->breadcrumbs(fn (Trail $trail) => $trail->parent('home') - ->push(trans('general.admin'), route('settings.index')));; + ->push(trans('general.admin'), route('settings.index'))); }); /*