Level 102
I don't see any constructor in that code
Oct 9, 2022
6
Level 9
Session is not accessible in Constructor
Kernel.php
protected $middleware = [
// \App\Http\Middleware\TrustHosts::class,
\App\Http\Middleware\TrustProxies::class,
\Fruitcake\Cors\HandleCors::class,
\App\Http\Middleware\PreventRequestsDuringMaintenance::class,
\Illuminate\Foundation\Http\Middleware\ValidatePostSize::class,
\App\Http\Middleware\TrimStrings::class,
\Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
];
/**
* The application's route middleware groups.
*
* @var array<string, array<int, class-string|string>>
*/
protected $middlewareGroups = [
'web' => [
\App\Http\Middleware\EncryptCookies::class,
\Illuminate\Cookie\Middleware\AddQueuedCookiesToResponse::class,
\Illuminate\Session\Middleware\StartSession::class,
// \Illuminate\Session\Middleware\AuthenticateSession::class,
\Illuminate\View\Middleware\ShareErrorsFromSession::class,
\App\Http\Middleware\VerifyCsrfToken::class,
\Illuminate\Routing\Middleware\SubstituteBindings::class,
],
'api' => [
// \Laravel\Sanctum\Http\Middleware\EnsureFrontendRequestsAreStateful::class,
'throttle:api',
\Illuminate\Routing\Middleware\SubstituteBindings::class,
],
];
web.php
Route::group(['middleware' => ['web']], function()
{
Route::get('/login', [LoginController::class, 'index'])->name('login');
Route::post('/authenticate', [LoginController::class, 'authenticate'])->name('authenticate');
Route::get('/logout', [LoginController::class, 'logout'])->name('logout');
});
Dasbhoard.php
public function __construct(){
if(CommonFunctionsHelper::chkLoggedIn() == null) {
Redirect::to('login')->with('message', 'Login to continue')->send();
}
}
CommonFunctionsHelper.php
public static function chkLoggedIn()
{
$authUser = Session::get('authUser');
if(!empty($authUser)){
return $authUser;
}
else{
return null;
}
}
LoginController.php
public function authenticate(LoginRequest $request)
{
Session::put('authUser',(array)$api_response->UserDetails);
return redirect()->route('dashboard');
}
}
Level 9
@Sinnbeck I have updated the code, can you check once again?
Level 102
What you are doing in the constructor is a job for middleware. Move the logic to a new middleware
Level 9
@Sinnbeck After writing new middleware where do I have to register?
protected $routeMiddleware = [
'auth' => \App\Http\Middleware\Authenticate::class,
'auth.basic' => \Illuminate\Auth\Middleware\AuthenticateWithBasicAuth::class,
'cache.headers' => \Illuminate\Http\Middleware\SetCacheHeaders::class,
'can' => \Illuminate\Auth\Middleware\Authorize::class,
'guest' => \App\Http\Middleware\RedirectIfAuthenticated::class,
'password.confirm' => \Illuminate\Auth\Middleware\RequirePassword::class,
'signed' => \Illuminate\Routing\Middleware\ValidateSignature::class,
'throttle' => \Illuminate\Routing\Middleware\ThrottleRequests::class,
'verified' => \Illuminate\Auth\Middleware\EnsureEmailIsVerified::class,
];
here?
Or can you provide any snippet?
Level 102
@bigrattle you just attach it directly to the route in web.php
->middleware('your_middleware')
You can add it to that list if you want to give it a short name to use in routes (recommended)
Level 9
@Sinnbeck thanks, it works.
Please or to participate in this conversation.