Комментарии к публикации
Пишем систему авторизации на PHP

Можешь кинуть код реализации ?

Header:

<td colspan="2" style="text-align: right">
            <?php if(!empty($user)): ?>
            Привет, <?= $user->getNickname() ?>  | <a href="http://myproject.loc/users/logOut">Выйти</a>
            <?php else: ?>
            <a href="http://myproject.loc/users/login">Войти</a> | <a href="http://myproject.loc/users/register">Зарегестрироваться</a>
            <? endif; ?>
        </td>

UsersController:

public function logOut()
    {
        setcookie('token', '', -1, '/', '', false, true);
        header('Location: /');
    }

routes:

'~^users/logOut~' => [\MyProject\Controllers\UsersController::class, 'logOut'],

header.php
<tr>
        <td colspan="2" style="text-align: right">
            <?= !empty($user) ? 'Привет, ' . $user->getNickname() . ' | <a href = " ../users/exit">Выйти</a>' : '<a href="../users/login">Войдите на сайт</a> | <a href="../users/register">Зарегистрироваться</a>' ?>
        </td>
    </tr>

UsersContoroller.php

public function exit()
{
    setcookie('token', $token, 0, '/', '', false, true);
    header('Location: /');
}

ну и конечно роут '~^users/exit~' => [\MyProject\Controllers\UsersController::class, 'exit'],

После успешного логина почему то куки принимают вид:
Set-Cookie: token=3%3A; path=/; HttpOnly
Почему токен так задается не правильно? Из-за этого у меня не определяет юзера и соответственно не работает ничего.
class UsersAuthService:

 /**
     * @param User $user
     */
    public static function createToken(User $user)
    {
        $token = $user->getId() . ':' . $user->refreshAuthToken();
        setcookie('token', $token, 0, '/', '', false, true);
    }

    /**
     * @return User
     */
    public static function getUserByToken(): ?User
    {
        $token = $_COOKIE['token'] ?? '';

        if (empty($token)) {
            return null;
        }

        [$userId, $authToken] = explode(':', $token, 2);

        $user = User::getById((int) $userId);

        if ($user === null) {
            return null;
        }

        if ($user->getAuthToken() !== $authToken) {
            return null;
        }

        return $user;
    }

Models/User.php:

/**
     * @param array $loginData
     * @return User
     * @throws InvalidArgumentException
     */
    public static function login(array $loginData): User
    {
        if (empty($loginData['email'])) {
            throw new InvalidArgumentException('Enter email');
        }
        if (empty($loginData['password'])) {
            throw new InvalidArgumentException('Enter password');
        }
        $user = User::findOneByColumn('email', $loginData['email']);
        if ($user === null) {
            throw new InvalidArgumentException('Can\'t find user with this email');
        }
        if (!password_verify($loginData['password'], $user->getPasswordHash())) {
            throw new InvalidArgumentException('Incorrect password!');
        }
        /*if (!$user->isConfirmed) {
            throw new InvalidArgumentException('Пользователь не подтверждён');
        }*/

        $user->refreshAuthToken();
        $user->save();

        return $user;
    }

    /**
     * @return string
     */
    public function getPasswordHash(): string
    {
        return $this->passwordHash;
    }

    public function refreshAuthToken()
    {
        $this->authToken = sha1(random_bytes(100)) . sha1(random_bytes(100));
    }

    public function getAuthToken()
    {
        return $this->authToken;
    }

UsersController:

public function login()
    {
        if (!empty($_POST)) {
            try {
                $user = User::login($_POST);
                UsersAuthService::createToken($user);
                header('Location: /');
                exit();
            } catch (InvalidArgumentException $e) {
                $this->view->renderHtml('users/login.php', ['error' => $e->getMessage()]);
                return;
            }
        }

        $this->view->renderHtml('users/login.php');
    }

Добрый вечер! решение д/з:

header.php

<?php if (!empty($user)) { ?>
    Привет, <?= $user->getNickname() ?> | <a href="/users/logout">Выйти</a>
<?php } else { ?>
    <a href="/users/login">Войдите на сайт</a> | <a href="/users/register">Зарегестрируйтесь</a>
<?php } ?>

UsersAuthService.php

public static function deleteToken()
{
    setcookie('token', '', false, '/', '', false, true);
}

UsersController.php

public function logOut()
{
    UsersAuthService::deleteToken();
    header('Location: /');
}

Роут

'~^users/logout~' => [\MyProject\Controllers\UsersController::class, 'logOut']

Метод

public function logout()
    {
        setcookie('token', $token, -1, '/', '', false, true);
        header('Location: /');
    }

Шапка

<td colspan="2" style="text-align: right"> 
            <?= !empty($user) ? 'Привет, ' . $user->getNickname() .
            ' | <a href="/users/logout"> Выйти</a>' :
             '<a href="/users/login">Войти</a> |
              <a href="/users/register">Зарегистрироваться</a>' ?>
        </td>

РОут

 '~^users/logout~' => [\MyProject\Controllers\UsersController::class, 'logout'],

Router: '~^users/logout~' => [UsersController::class, 'logout']

UsersController:

public function logout()
    {
        try {
            if ($this->user !== null) {
                if (User::logout()) {
                    header('Location: /');
                    exit();
                }
            } throw new AuthException('Вы не зарегистрированы', 403);
        } catch (AuthException $e){
            $this->view->renderHtml('/errors/403.php', ['error' => $e->getMessage()], 403);
        }
    }
User Model:

public static function logout(): bool
    {
        if (isset($_COOKIE['token'])) {
            setcookie("token", "", time() - 1000, "/");
            return true;
        }
        return false;
    }

routes.php

'~logout~' => [\MyProject\Controllers\UsersController::class, 'logout'],

header.php

<? if(!empty($user)): ?>
                <?= 'Привет, ' . $user->getNickname()?> | <a href="/logout">Выйти</a>
            <?php else: ?>
                <a href="/users/login">Войти</a> | <a href="/users/register">Зарегистрироваться</a>
            <?php endif; ?>

User.php

public function logout()
    {
        $this->logout();
    }

UsersController.php

public function logout() // разлогинивание пользователя
    {
        setcookie('token', time()-1);
        header('Location: /');
    }

routes.php:

'~^logout$~' => [\MyProject\Controllers\UsersController::class, 'logout'],

index.php:

<?php if (!empty($user)): ?>
    Привет, <?= $user->getNickname(); ?> |
    <a href="/logout">Выйти</a>
<?php else: ?>
    <a href="/users/login">Войти</a> |
    <a href="/users/register">Зарегистрироваться</a>
<?php endif; ?>

UsersAuthService.php

    public static function deleteCookie(): void
    {
        setcookie('token', '', -1, '/', '', false, true);
    }

UsersController.php

    public function logout()
    {
        UsersAuthService::deleteCookie();
        header('Location: /');
    }

Не знаю, правильно ли я сделал, что в UsersAuthService переименовал метод createToken на saveCookie и добавил второй метод deleteCookie, вместо deleteToken. Мне почему-то показалось так понятнее, ведь мы создаем новый токен при авторизации и сохраняем его в БД. А в методе createToken мы всего лишь берем его из БД и устанавливаем куки. Допустимо ли так делать, или лучше оставить как было?

Расскажите, пожалуйста, что представляют из себя "сервисы" в целом.
Какие ещё сервисы могут быть в типовых проектах?
И как понять, в какой ситуации лучше создавать сервис?

Артём, здравствуйте!
Возник вопрос.
Можете рассказать, как лучше организовать восстановление пароля?
Если требуется проверить данные формы(что они корректны), найти аккаунт в БД и если он существует, сгенерировать новый пароль и отправить его на почту.
Вопрос в том, что и где должно быть и как взаимодействовать.

Спасибо за урок!

UsersController.php:

public function logout(): void
    {
        setcookie('token', '', 1, '/');
        header('Location: /');
    }

templates/header.php:

<?php
            if (!empty($user)): ?>
                Привет, <?= $user->getNickname() ?> | <a href="/users/logout">Выйти</a>
            <?php else: ?>
                <a href="/users/login">Войти</a> | <a href="/users/register">Зарегистрироваться</a>
            <?php endif; ?>

routes.php

...
'~^users/logout$~' => [UsersController::class, 'logout']
...

UsersController.php

public function logout (): void
    {
        UsersAuthService::deleteToken();
        header('Location: /');
        exit;
    }

UsersAuthService.php

public static function deleteToken(): void
    {
        $token = $_COOKIE['token'];
        setcookie('token', $token, 1, '/', '', false, true);
    }

header.php

<td colspan="2" style="text-align: right">
            <?= !empty($user) ? 'Hello, ' . $user->getNickname().' | <a href="/users/logout">Log out</a>' : '<a href="/users/login">Log in</a> | <a href="/users/register">Sign Up</a>' ?>
        </td>

Традиционно вопросы:
1.
Почему файл сервиса активирования лежит в папке user, а файл сервиса авторизации в папке services? Может их лучше в одном месте складировать - в той же папке сервисы?

2. При обработке исключений, как лучше выводить сообщения? Если в рамках этого учебного проекта - в шаблоне, в части main, или просто отдельным сообщением без всякой разметки? Нужны ли в этом случае ссылки, например перехода на главную? Или это ситуация исключительная, поэтому и выход из нее должен осуществляться кнопкой назад в браузере или переходом через адресную строку? Или это дело вкуса/хорошего тона?

3.
Аналогично предыдущему уроку - некая путаница в названиях, сильно с толку сбивает такая схожесть в названиях. И для регистрации-активации токены и для авторизации через куки - тоже токены..

4.
Если нужно запомнить пользователя на дольше, чем время работы в браузере, для каждого пользователя нужно делать две куки - для стандартного(на время работы браузера) и для того, чтобы запомнить на дольше?
Читал про использование сессий для авторизации, в т.ч. и относительно вопросов безопасности, пока не осилил, в частности, непонятна прелесть использования сессий для авторизации в контексте того, что время сессии ~20 минут, т.е. каждые 20 минут нужно перелогиниваться, если нет куки на более длительное время?

5.
Скорее риторическая ремарка... В целом - к сожалению, несмотря на то, что я нахожу причину любой ошибки, которую сделал, не проблема разобраться в любой части проекта и в реализованых взаимосвязях между ними, все еще сложно охватить пониманием всю структуру - если бы это делать самостоятельно и с нуля - что разместить в контроллере, что во фронт-контроллере, что в сервисе, что в модели...

header.php

 <td colspan="2" style="text-align: right">
            <? if (!empty($user)) { ?>
                Привет,  <?=$user->getNickname()?> <a href=/users/logout>Выйти</a>
            <? } else { ?>
                <a href="/users/login">Войдите на сайт</a>
                <a href="/users/register">Регистрация</a>
            <? } ?>
        </td>

UserController.php

 public function logout() {
        User::logout();
}

User.php

 public static function logout() {
        setcookie('token', '', -1, '/', '', false, true);
        header('Location: /');
}

Я не понял, почему не работает setcookie('token', '', -1)

//header.php
...
 <td colspan="2" style="text-align: right">
            <?= !empty($user) ? 'Привет, ' . $user->getNickname() .'|<a href="http://myproject.loc/users/logout">Выйти</a>': '<a href="http://myproject.loc/users/login">Войти</a>|<a href="http://myproject.loc/users/register">Зарегистрироваться</a>' ?>
        </td>
...

//UsersAuthService.php
...
public static function deleteToken(){
        setcookie('token', '', 0, '/', '', false, true);
    }
...

//UserController
...
public function logout(){
        if(!empty($_COOKIE)){
            UsersAuthService::deleteToken();
            header('Location: /');
            exit();
        }
    }
...

Домашка

routes

'~^users/logout$~' => [\MyProject\Controllers\UsersController::class, 'logout']

UsersController

  public function logout()
  {
    setcookie('token', '', -10, '/', '', false, true);
    header('Location: /');
  }

header.php

<td colspan="2" style="text-align: right">
        <?php if(!empty($user)): ?>
          Привет <?= $user->getNickname() ?>
          <a href="/users/logout">Выйти</a>
        <?php else: ?>
          <a href="/users/login">Войти</a>
          <a href="/users/register">Зарегистрироваться</a>
        <?php endif;?>
      </td>

routes.php:

...
'~^users/exit~' => [\MyProject\Controllers\UsersController::class, 'exit'],
...

UsersController.php:

...
public function exit()
    {
        $user = UsersAuthService::getUserByToken();
        if (!empty($user)) {
            $nick = $user->getNickname() ?? '';
        };
        if (isset($_COOKIE['token'])) {
            setcookie('token', '', time() - 3600, '/', '', false, true);
        }
        unset($user);
        $this->view->renderHtml('users/exit.php', ['nick' => $nick, 'user' => null]);
    }

...

exit.php:

<?php include __DIR__ . '/../header.php'; ?>
    <div style="text-align: center;">
        <h2>До свидания <?= $nick ?>, заходите еще!</h2>
        <a href="/">Вернуться на главную страницу</a>
    </div>
<?php include __DIR__ . '/../footer.php'; ?>

header.php:

...
<tr>
        <td colspan="2" style="text-align: right">
            <?= !empty($user) ? 'Привет, ' . $user->getNickname().
                ' | <a href="/users/exit">Выйти</a>':
                '<a href="/users/login">Войти</a> | <a href="/users/register">Зарегистрироваться</a>'?>
        </td>
    </tr>
...

header.php

        <td colspan="2" style="text-align:right">
            <?= !empty($user) ? 'Привет, '.$user->getNickname().' | <a href="/users/logout">Выйти</a>' :
                '<a href="/users/login">Войти на сайт</a> | <a href="/users/register">Зарегистрироваться</a>' ?>
        </td>

routes.php

    '~^users/logout~'=>[\MyProject\Controllers\UsersController::class, 'logout'],

UsersController

    public function logOut()
    {
        setcookie('token', null, -1, '/','', false, true);
        header('Location: /users/login');
    }

header

<td  colspan="2" style="text-align: right">
                <?php if (!empty($user)): ?>
                    Привет <b><?= $user->getNickName() ?></b> <br>
                    <a href="users/logout">Выход</a>
                <?php else: ?>
                    <a href="users/login">Вход</a>
                <?php endif ?>

            </td>

UsersController

 public function logout()
    {
        if (isset($_COOKIE['token']))
        {
            setcookie ("token", "", -1 , '/', '', false, true);
            header('Location: /');
        }else
            {
                header('Location: /');
            }
    }

routes

'~^users/logout$~' => [\controllers\UsersController::class, 'logout'],

templates\header.php

...
    <tr>
        <td colspan="2" style="text-align: right">
            <?php
            if (!empty($user)) {
                echo 'Привет, ' . $user->getNickname() . ' | <a href="/users/logout">Выйти</a>';
            }else {
                echo '<a href="/users/login">Войти</a> | <a href="users/register">Зарегистрироваться</a>';
            }
            ?>
        </td>
    </tr>
...

src\routes.php

...
    '~^users/logout$~' => [\MyProject\Controllers\UserController::class, 'logout'],
...

src\MyProject\Controllers\UserController.php

...
    public function logout()
    {
        if (!empty($this->user)) {
            setcookie('token', '', -1, '/', '', false, true);
        }
        header('Location: /');
        exit();
    }
...

А что значит такая запись?

[$userId, $authToken] = explode(':', $token, 2);

И гуглил, и урок по массивам посмотрел. Нигде не видел такого. Функция explode возвращает массив, это понятно. А вот что слева непонятно. Буду благодарен.

header.php

<?= !empty($user) ? 'Привет, ' . $user->getNickname() . ' | <a href="/users/unlogin">Выйти</a>': '<a href="/users/login">Войти</a> | <a href="/users/register">Зарегистрироваться</a>' ?>

routes.php

'~^users/unlogin~' => [\MyProject\Controllers\UsersController::class, 'unlogin'],

UsersController.php

    public function unlogin() {
        setcookie('token', '', -1);
        if ($_SERVER['HTTP_REFERER']) {
            header('Location: ' . $_SERVER['HTTP_REFERER']);
        }
        else {
            header('Location: /');
        }
    }

Почему не разлогинивает?

Если пользователь залогинен, то сделайте рядом с приветствием ссылку для выхода из этой учетной записи.

Если пользователь не залогинен, показывайте ссылки для регистрации и входа на сайт.

header.php

...
<td colspan="2" style="text-align: right">
            <?php if (!empty($user)): ?>
                Привет, <?= $user->getNickname() ?> | <a href="/users/logout">Выйти</a>
            <?php else: ?>
                <a href="/users/login">Войти</a> | <a href="/users/register">Зарегистрироваться</a>
            <?php endif; ?>
        </td>
...

Реализуйте функционал для разлогинивания (роутинг + экшен, удаляющий cookie).

routes.php

...
'~^users/logout~' => [\MyProject\Controllers\UsersController::class, 'logout'],
...

UsersController.php

...
    public function logout(): void
    {
        if ($this->user !== null) {
            UsersAuthService::logout();
        }

        $this->view->renderHtml('users/logout.php');
        return;
    }
...

UsersAuthService.php

...
    public static function logout(): void
    {
        setcookie('token', '', 1, '/');
        return;
    }
...

templates/logout.php

<h1>Вы вышли.</h1>
<a href="/"><h2>Вернуться на главную</h2></a>

Долго думал как поступить с методом удаления куки, реализовал его в UsersAuthService. Ведь контроллеры отвечают за передачу данных в модель и сами не изменяют их. А в UsersAuthService мы создавали куки там, и там же их и проверяем, так почему бы там их и не удалять. Вся работа с куки будет только в одном файле)

1.
Почему в роуте

'~^users/login~'

нет экранирования слэша? Работает и так, но regex101 выделяет это как ошибки.

2.
Почему метод refreshAuthToken публичный, а не приватный?

3.
почему catch в методе login после exit, а не после User::login? После User::login ловить же уже нечего

4.
Как продолжение предыдущего вопроса: Какой вариант предпочтительнее, есть ли вообще принципиальная разница между следующими вариантами:

В юзер-контроллере:

/**
     * @throws ForbiddenException
     */
    public function logout()
    {
        UserAuthService::tokenDelete();
        header('Location: /');
    }

В UserAuthService:

/**
     * @throws ForbiddenException
     */
    public static function tokenDelete()
    {
        if (!isset($_COOKIE['token'])) {
            throw new ForbiddenException('Unauthorized access');
        }
        setcookie('token', '', 0, '/', '', false, true);
    }

Ловить исключение во фронт- контроллере или в юзер-контроллере? Или вообще выкидывать исключение в юзер-контроллере, а ловить во фронте?

Как это представляю себе я: Ловить исключение во фронт-контроллере видится более универсальным, а выбрасывать в юзер-контроллере или в классе сервисов, мне кажется - без разницы. Сделал - выбрасывание сразу на этапе возможного возникновения ошибки, мне так видится более логичным.

5.
Наверное тоже туда до кучи вопрос - про идентификацию исключений/код ошибки - если была сделана попытка несанкционированного доступа - например из этого же урока - удаление куки неавторизованным пользователем - это 403 или 401?

6.

Теперь нам нужно научиться передавать пользователя во View.

А что, если его туда не передавать, а прямо там и получать, если он существует? Например вот так:

клас View:

<?php

namespace MyProject\View;

use MyProject\Models\Users\UserAuthService;

/**
 * Class View
 * @package MyProject\View
 */
class View
{
    /**
     * @var string
     */
    private $templatePath;

    /**
     * @var
     */
    private $userFromToken;

    /**
     * View constructor.
     * @param string $templatePath
     */
    public function __construct(string $templatePath)
    {
        $this->templatePath = $templatePath;
        $this->userFromToken = UserAuthService::getByToken();

    }

    /**
     * @param string $templateName
     * @param array $vars
     * @param int $code
     */
    public function renderHtml(string $templateName, array $vars = [], $code = 200)
    {
        $user = $this->userFromToken;
        http_response_code($code);
        extract($vars);
        ob_start();
        include_once $this->templatePath . '/' . $templateName;
        $buffer = ob_get_contents();
        ob_end_clean();
        echo $buffer;
    }
}

Где находится сам метод getAuthToken()?
PHP ругается из за отсутствие этого метода.
Может где-то я пропустил, так и не смог найти. :/

//Route
'~^users/logout$~' => [\MyProject\Controllers\UsersController::class, 'logout']

//Action
 public function logout()
        {
            setcookie('token', '', 0, '/');
            header('Location: /');
            return;
        }

//Header
<td colspan="2" style="text-align: right">
            <?= !empty($user) ?
                'Привет, ' . $user->getNickname() . ' | <a href="/../users/logout">Logout</a>' :
                '<a href="/../users/login">Login in</a> | <a href="/../users/register">Sign Up</a>' ?>
        </td>

?

routes

'~^users/logout$~' => [\MyProject\Controllers\UsersController::class, 'logout']

header

<?= !empty($user) ? 'Привет, ' . $user->getNickname() . ' | <a href = "/test/users/logout">Выйти</a>' : '<a href = "/test/users/login">Войти</a> | <a href="/test/users/register">Зарегистрироваться </a>' ?>

UsersController

public function logout()
    {
        setcookie('token', '', 0, '/', '', false, true);
        header('Location: /test/index.php');
    }

домашка

header.php

<td colspan="2" style="text-align: right">
            <?= !empty($user) ? 'Привет, ' . $user->getNickName() . ' <a href="/phplearn.my/www/users/logout">Выйти</a>' : '<a href="/phplearn.my/www/users/login">Войти</a> | <a href="/phplearn.my/www/users/register">Зарегистрироваться</a>' ?>
        </td>

routes.php

'~^users/logout~' => [\MyProject\Controllers\UsersController::class, 'logout']

UsersController.php

use MyProject\Models\Users\User;
use MyProject\Services\UsersAuthService;
...
    public function logout()
    {
        UsersAuthService::deleteToken();
        header('Location: /phplearn.my/www/');
        exit();
    }

UsersAuthService.php

    public static function deleteToken(): void
    {
        $token = null;
        setcookie('token', $token, 0, '/', '', false, true);
    }

роут

'~^users/logout~' => [\MyProject\Controllers\UsersController::class, 'logout']

UsersAuthService

public static function deleteToken()
    {
        setcookie('token', '', false, '/', '', false, true);
    }

контроллер

 public function logout()
    {

        UsersAuthService::deleteToken();
        header('Location: /');
    }

У меня есть вопрос, общий. А можно ли в представлении после ob_start сразу же подключать layouts header and footer, чтобы каждый раз в новом шаблоне не подкючать эти компоненты?Если нет - пожалуйста, обьясните.Спасибо!

роутер

    '~^users/logout$~' => [MyProject\Controllers\UsersController::class, 'logout'],

контроллер

public function logout ()
    {
        if (!empty($this->user)) {
            $this->user->logout();
        };
    header('Location: /');
    exit();
    }

модель

public function logout() {
    UsersAuthService::deleteToken();
}

сервис

public static function deleteToken () : void
     {
         setcookie('token', '', -1, '/', '', false, true);
     }

routes.php

'~^user/logout$~'=>[\MyProject\Controllers\UsersController::class, 'logout'],

UsersController.php

public function logout()
    {
        setcookie('token', '', -1, '/', '', false, true);
        header('Location: /');
    }

header.php

 <td colspan="2" style="text-align: right">
            <?php if(!empty($user)): ?>
            Привет, <?= $user->getNickname() ?> | <a href="http://study/users/logout">Выйти</a>
            <?php else: ?>
            <a href="http://study/users/login">Войти</a> | <a href="http://study/users/register">Зарегистрироваться</a>
            <? endif; ?>
        </td>

Моя попытка решить домашнее задание.

routes.php

................
    '~^users/logout/$~' => [\MyProject\Controllers\UsersController::class, 'logout'],
................

UsersController.php

    public function logout()
    {
        /*
            1 проверка куки
                есть - метод выхода в сервисе
                нет - отображение шаблона, что вышли
        */

        $token = $_COOKIE['token'] ?? '';

        if (!empty($token)) {

            UsersAuthService::deleteToken();
            //Переход на главную страницу, хотя д.б. наверное на стра авторизации или ту с которой запрос "выйти"
            header('Location: /');
            return;
        }

        // Если уже не авторизованный запрос пытаются разлогиниться - переход на прежнюю страницу
        header('Location: '.$_SERVER["HTTP_REFERER"]);
        return;
    }

    public function profile($params)
    {
        var_dump($params);
        var_dump($_SERVER);
    }

UsersAuthService.php

    public static function deleteToken(): void
    {
        // $token = $user->getId() . ':' . $user->getAuthToken();
        setcookie('token', "", time() - 3600, '/', '', false, true);
    }

header.php

<!DOCTYPE html>
<html lang="ru">
<head>
    <meta charset="UTF-8">
    <title>Мой блог</title>
    <link rel="stylesheet" href="/styles.css">
</head>
<body>

<table class="layout">
    <tr>
        <td colspan="2" class="header">
            Мой блог
        </td>
    </tr>
    <tr>
        <td colspan="2" style="text-align: right">

            <?php

                // Формирование ссылки на профиль, записи при авторизованной загрузке страницы
                if ($user!==Null)
                {
                    var_dump($user);
                    $user_profile_link_1  = "/users/profile/";//.$user->getId();
                    $user_profile_link_2  = "/users/logout/";//.$user->getId();
                    $logined = 'Привет, <a href="'.$user_profile_link_1.'">'. $user->getNickname().'</a> | <a href="'.$user_profile_link_2.'">Выйти</a>';

                }
                else{

                    // не авторизованная загрузка страница
                    $not_logined     =  '<a href="/users/login">Войти</a>';
                    $not_logined    .=  ' | <a href="/users/signup/">Зарегистрироваться</a>';
                }
            ?>

            <?= (!empty($user)||($user!==Null)) ? $logined:$not_logined;?>

        </td>
    </tr>
    <tr>
        <td>

Пришлось добавить проверку при авторизации что пользователь если уже авторизован то открыть страницу статьи.
Обнаруил такую уязвимость при попытке реализовать на уроке 32 комментарии

UsersController.php

.......................
    public function login()
    {
        if (!empty($_POST)) {
            try {
                $user = User::login($_POST);
                UsersAuthService::createToken($user);
                header('Location: /');
                exit();
            } catch (InvalidArgumentException $e) {
                $this->view->renderHtml('users/login.php', ['error' => $e->getMessage()]);
                return;
            }
        }

        if ($this->user!==null){
            header('Location: /');
            exit();
        }

        $this->view->renderHtml('users/login.php');
    }
....................