Как избавиться от ошибки Missing required parameters for [Route: password.reset] [URI: {locale}/password/reset/{token}]?

Question

[tol64]

Ошибка возникает при попытке сбросить пароль. На тестовых файлах фреймворка всё работает хорошо. Но после изменения маршрутов возникает ошибка при попытке сбросить пароль. На формах авторизации и регистрации всё нормально.

web.php

Route::group([
    'prefix' => '{locale}',
    'where' => ['locale' => '[a-zA-Z]{2}'],
    'middleware' => 'setlocale'
], function () {
    Route::get('/', function () {
        return view('index');
    });
    Auth::routes();
});

Результат

Illuminate\Routing\Exceptions\UrlGenerationException
Missing required parameters for [Route: password.reset] [URI: {locale}/password/reset/{token}].

Подскажите пожалуйста, как решить эту проблему. Понимаю так, что параметр {token} не передаётся, но как его передать в такой схеме, пока не знаю.

Comments

[JhaoDa]

но как его передать в такой схеме, пока не знаю

А знаешь, как вообще параметры в роут передаются?

[tol64]

JhaoDa, хочу узнать.
До внесения изменений на исходных тестовых файлах всё работает. После внесения перечисленных выше изменений, перестаёт. Можете объяснить, почему?

[JhaoDa]

tol64, т.е. вот эту строчку

<form method="POST" action="{{ route('password.email', app()->getLocale()) }}" class="auth_login">

изменил не ты? Или изменил, но не понял, что сделал?

хочу узнать

Так кто же мешает? Документацию начни читать.

[tol64]

JhaoDa, документацию читаю постоянно. Задаю вопрос только тогда, когда долго не получается решить задачу.

Указанную строчку изменил я. И это работает в описанной схеме для авторизации и регистрации юзера.

Вопросы:

1. Почему это не работает в случае сброса пароля?
2. Как правильно интерпретировать полученную ошибку?
3. Почему токен не обнаруживается?
4. Как правильно решить эту задачу?

[JhaoDa]

tol64, пардон, я чот сдуру в шаблон вцепился. Впрочем, причина проблема от этого не изменилась. Где в коде генерация роута на сброс пароля? Если раньше она выполнялась дефолтным способом, то теперь тебе надо делать это самому.

[tol64]

JhaoDa,

т.е. вот эту строчку ... изменил не ты? Или изменил, но не понял, что сделал?

В указанной Вами строчке вторым параметром передаётся значение, которое затем используется в качестве префикса в файле routes\web.php, как показано ниже:

Route::group([
    'prefix' => '{locale}',
    'where' => ['locale' => '[a-zA-Z]{2}'],
    'middleware' => 'setlocale'
], function () {
    Route::get('/', function () {
        return view('index');
    });
    Auth::routes();
});

Попробовал передать массивом, но тоже не получилось:

<form method="POST" action="{{ route('password.email', ['locale' => $locale, 'token' => csrf_token()]) }}" class="auth_login">

[tol64]

JhaoDa,

Где в коде генерация роута на сброс пароля? Если раньше она выполнялась дефолтным способом, то теперь тебе надо делать это самому.

В файле Routing\Router.php можно найти вот такой метод:

/**
     * Register the typical reset password routes for an application.
     *
     * @return void
     */
    public function resetPassword()
    {
        $this->get('password/reset', 'Auth\ForgotPasswordController@showLinkRequestForm')->name('password.request');
        $this->post('password/email', 'Auth\ForgotPasswordController@sendResetLinkEmail')->name('password.email');
        $this->get('password/reset/{token}', 'Auth\ResetPasswordController@showResetForm')->name('password.reset');
        $this->post('password/reset', 'Auth\ResetPasswordController@reset')->name('password.update');
    }

То есть из формы на сброс пароля переходим в метод sendResetLinkEmail() контроллера Auth\ForgotPasswordController

/**
     * Send a reset link to the given user.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\RedirectResponse|\Illuminate\Http\JsonResponse
     */
    public function sendResetLinkEmail(Request $request)
    {
        $this->validateEmail($request);
        // We will send the password reset link to this user. Once we have attempted
        // to send the link, we will examine the response then see the message we
        // need to show to the user. Finally, we'll send out a proper response.
        $response = $this->broker()->sendResetLink(
            $this->credentials($request)
        );
        return $response == Password::RESET_LINK_SENT
                    ? $this->sendResetLinkResponse($request, $response)
                    : $this->sendResetLinkFailedResponse($request, $response);
    }

И вот в этом методе, как раз и получается сбой.

Answer 1

[jazzus]

Т.к. ко всем auth роутам добавляется новая переменная locale ее нужно передавать во всех роутах первым аргументом, иначе будут ошибки. В этом случае в нотификации ResetPassword в строке

->action(Lang::get('Reset Password'), url(config('app.url').route('password.reset', ['en', 'token' => $this->token, 'email' => $notifiable->getEmailForPasswordReset()], false)))

В вендоре менять ничего нельзя. Нужны свои файлы и методы, как это делать в гугле куча материала.
Нужно пройтись по всем роутам, искать по name роута (например здесь 'password.reset') и добавить локаль первым аргументом.

Comments

[tol64]

К сожалению не понял Ваш ответ.

Я не нашёл указанной Вами строки кода в vendor\laravel\framework\src\Illuminate\Foundation\Auth\ResetsPasswords.php

А сейчас разве переменная locale не первым аргументом передаётся в качестве префикса для всех роутов?

Какие нужно создать файлы?

Методы нужно переопределить в контроллерах в папке app\Http\Controllers\Auth ?

• ConfirmPasswordController.php
  
• ForgotPasswordController.php
  
• LoginController.php
  
• RegisterController.php
  
• ResetPasswordController.php
  
• VerificationController.php
  

Список роутов:
Чтобы посмотреть в полном размере, правой кнопкой по картинке и открыть в новой вкладке.


Можете дать ссылку, где можно подробнее об этом прочитать? Я пересмотрел очень много материалов и попробовал очень много вариантов, но пока не получилось решить эту задачу. Где-то недопонимаю, пытаюсь понять где.

Может быть есть, какая-то простая возможность, переопределить всего лишь один маршрут?

GET|HEAD 
{locale}/password/reset/{token}
password.reset
App\Http\Controllers\Auth\ResetPasswordController@showResetForm
web

[jazzus]

tol64, я же написал как это все отредактировать. Нужно искать по name всех auth роутов. В данном случае 'password.reset'. Нотификация ResetPassword где этот роут, находится в Illuminate\Auth\Notifications. Там где роут нужно отправить локаль первым аргументом, как в моем примере. Как переопределять отдельная тема. Для теста можно в вендоре сейчас зименить потом вернуть к дефолту

[tol64]

jazzus, уже нашёл. Спасибо. Не там смотрел изначально.

На другом форуме (Laracast) подсказали более точнее. Для моего случая оказалось достаточно сделать префикс необязательным (?) параметром:

'prefix' => '{locale?}'

После этого ошибка исчезла и письмо отправилось на email пользователя указанный в форме.

А вот при нажатии на кнопку Reset Password в самом письме, переход осуществлялся на несуществующую страницу (404).

Для этого случая воспользовался Вашим решением. Спасибо.

Что касается переопределения метода для отправки письма, то попробую пока сам решить. Вопрос актуальный, так как нужна возможность изменять текст и оформление письма.

Answer 2

[tol64]

Для того, чтобы ошибка не проявлялась при описанной схеме, достаточно сделать необязательным (?) префикс-параметр, а также перечислить языки регулярным выражением, как в листинге ниже:

Route::group([
    'prefix' => '{locale?}',
    'where' => ['locale' => '^ru|en$']
    'middleware' => 'setlocale'
], function () {
    Route::get('/', function() {
        return view('index');
    });
    Auth::routes();
});

Comments

[jazzus]

Писец. Где в вопросе, что локаль может быть необязательна? что будет website/login работать это норм? Мое решение как раз по твоему вопросу, а твой вариант это вообще другая логика.

[tol64]

jazzus, не локаль необязательна, а параметр. Вы пробовали протестировать?

В описанной схеме вариант website/login не пройдёт. Будет 404.

Просто языки нужно прописать. Например, в where регулярка может быть такой:

'where' => ['locale' => '^ru|en$']

Добавил в ответ.

В итоге, например, такие адреса будут доступны:

/
/ru
/en
/ru/login
/en/login

А вот такие нет (404):

/de
/de/login

[jazzus]

tol64, ты использовал мое решение? Если ты уберешь знак вопроса из префикса и оставишь только аргумент в ResetPassword ошибка останется?

[tol64]

jazzus, Ваш вариант тоже избавляет от ошибки. Спасибо. Но нужно решение без внесения изменений глубоко в коде фреймворка.

На сколько понял нужно переопределить метод sendPasswordResetNotification() в vendor\laravel\framework\src\Illuminate\Auth\Passwords\CanResetPassword.php.

И сделать это можно в app\User.php (хитрая система наследований).

[jazzus]

tol64, мой вариант тоже работает или ты его использовал вместе со своим знаком вопроса?

[tol64]

jazzus, да, работает без знака вопроса.

[jazzus]

tol64, я не спрашиваю работает или нет. Я спрашиваю используешь ли ты мое решение или нет? Ты написал что не подходит т.к. нужно лезть в вендор. Знак вопроса в locale избавило тебя от этой необходимости?

[tol64]

jazzus, что касается ошибки, указанной в вопросе этой темы, то, да, знак вопроса в locale, как указание этого префикс-параметра в качестве необязательного, избавляет от необходимости лезть в вендор.

Но необходимость переопределить метод для отправки письма для сброса пароля всё равно остаётся (не в вендоре).

1. Ваше изменение для вендора решает проблему перехода из письма на страницу изменения пароля. Без этого в письме в кнопке для перехода (Reset password) содержится ссылка без учёта локали. Переход осуществляется на несуществующую страницу (404).
2. По каким-то причинам, даже с Вашим исправлением, изменить пароль после успешного перехода, не получается. Токен оказывается недействительным. Сейчас пытаюсь разобраться с этим вопросом. Создам новый проект и потестирую его без указанных изменений.

[jazzus]

А отключить нотификацию пробовал? Или верификацию емейл или юзеров? Это все поможет исправить ошибку.

необходимость переопределить метод для отправки письма для сброса пароля всё равно остаётся

Не необходимость останется, а в данный момент ты используешь мое решение и сам не знаешь зачем поставил знак вопроса в префикс и выбрал его ответом

[tol64]

jazzus, зачем отключать нотификацию и верификацию, если всё это нужно?

Не необходимость останется, а в данный момент ты используешь мое решение и сам не знаешь зачем поставил знак вопроса в префикс и выбрал его ответом

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

В любом случае, все данные для воспроизведения приведены. Каждый сам может попробовать провести все необходимые тесты.

[jazzus]

tol64, почему ты не признаешься, что ты в данный момент используешь мое решение? Зачем ты выкручиваешься? Конечно в вендоре не нужно ничего менять. Я тебе об этом два раза написал. Ты не знаешь, как не менять вендор поэтому сейчас исправил в вендоре. А как узнаешь сделаешь в своей нотификации. Т.е. и сейчас используешь мое решение и будешь использовать в будущем.

Ваш вариант тоже избавляет от ошибки. Спасибо

да, работает без знака вопроса.

изменения в вендоре не подходит для решения проблемы

[tol64]

jazzus, не вижу, каких либо противоречий со своей стороны. Да, Ваше решение не подходит, по той причине, что нужно внести изменения в вендоре. Поэтому я его не использую и не буду использовать в продакшене. К тому же тесты показали, что есть другие проблемы и с Вашим решением.

А в собственной нотификации решение будет совсем другим.

Что касается выбранного ответа. То выбран тот ответ, который избавляет от указанной ошибки и не требует изменений в вендоре. Всё вполне обосновано. Но убедить себя можно конечно во всём, что угодно. Как будто заняться больше нечем. Странно, что для Вас это так принципиально.

[jazzus]

tol64, ты в данный момент используешь мое решение или нет?

[tol64]

jazzus, нет. Я использую то, что выбрано в качестве ответа.

[jazzus]

tol64, и роут password.reset работает без локали?

[tol64]

jazzus, да.

Письмо отправляется по указанному в форме email.

Ошибки Missing required parameters for [Route: password.reset] [URI: {locale}/password/reset/{token}] не возникает.

[jazzus]

tol64, что будет, когда ты нажмешь на Reset Password?

[tol64]

@jazzus, что будет, когда ты нажмешь на Reset Password?

К данному вопросу это уже не имеет отношения. Интересовало только, как избавиться от указанной в вопросе ошибки. Все последующие проблемы предполагалось решать отдельно.

[jazzus]

К данному вопросу это уже не имеет отношения.

значит роут password.reset не работает. А зачем пишешь что работает? Или мы не роут password.reset здесь обсуждаем?

Интересовало только, как избавиться от указанной в вопросе ошибки.

Любым способом? Я выше предложил варианты

[tol64]

@jazzus, значит роут password.reset не работает. А зачем пишешь что работает? Или мы не роут password.reset здесь обсуждаем?

Обсуждаем проблему, которая относится к ошибке Missing required parameters for [Route: password.reset] [URI: {locale}/password/reset/{token}]

Любым способом? Я выше предложил варианты

Спасибо за варианты. Всё это в любом случае полезно и интересно. По крайней мере мне. Ваш вариант рассматриваю сейчас более подробно. Это можно будет обсудить в отдельной теме, если возникнут проблемы с решением.

[jazzus]

tol64, ошибка не исправлена, а закрывается твоим костылем во вьюхе. Как был битый роут password.reset, так и остался. Мое решение исправляет эту ошибку и во вьюхе и в письме. Формирует правильный роут. С помощью которого можно изменить пароль. Токен у тебя далее не определяется т.к. он ждет его первым аргументом, где у тебя локаль. Идешь по роуту далее и исправляешь

[tol64]

jazzus, Вы правы.

@jazzus, Токен у тебя далее не определяется т.к. он ждет его первым аргументом, где у тебя локаль. Идешь по роуту далее и исправляешь

Вот этот момент можете точнее подсказать? Где именно (в коде) токен считывается в данном случае, как не первый аргумент?

[jazzus]

tol64, добавить

public function showResetForm(Request $request, $locale, $token = null)
{
    return view('auth.passwords.reset')->with(
        ['token' => $token, 'email' => $request->email]
    );
}

В ResetPasswordController

[tol64]

jazzus, спасибо. Я тоже достиг рабочего варианта с токеном, но немного иначе.

Получил токен в шаблоне через request:

<input type="hidden" name="token" value="{{ request()->token }}">

Но Ваш вариант мне нравится больше, поэтому буду использовать его. Спасибо.

Ещё, чтобы не менять ничего в вендоре, я переопределил метод, в котором Вы порекомендовали ранее временно добавить локаль. В app\Providers\AppServiceProvider.php добавил такой код:

/**
     * Bootstrap any application services.
     *
     * @return void
     */
    public function boot()
    {
        self::setMailUsing();
    }
    /**
     * Set a callback (override) that should be used when building the notification mail message.
     *
     * @return void
     */
    public function setMailUsing() {
        ResetPassword::toMailUsing(function ($notifiable, $token) {
            return (new MailMessage)
                ->subject(Lang::get('reset.mail_subject'))
                ->greeting(Lang::get('reset.mail_greeting'))
                ->line(Lang::get('reset.mail_line1'))
                ->action(Lang::get('reset.mail_reset_action'), url(config('app.url')
                    .route('password.reset', [
                            'locale' => app()->getLocale(),
                            'token' => $token, 
                            'email' => $notifiable->getEmailForPasswordReset()
                            ], 
                            false)))
                ->line(Lang::get('reset.mail_line2', 
                    ['count' => config('auth.passwords.'.config('auth.defaults.passwords').'.expire')]))
                ->line(Lang::get('reset.mail_line3'))
                ->salutation(Lang::get('reset.mail_salutation'));
        });
    }

Теперь учитываются переводы на все указанные языки и всё работает без сбоев.

[jazzus]

tol64, как кастомизировать reset password написано в доках где вместо ResetPasswordNotification можно добавить свою нотификацию

[tol64]

@jazzus, как кастомизировать reset password написано в доках где вместо ResetPasswordNotification можно добавить свою нотификацию

Читал подробнее об этом варианте вот в этой статье:
Изменение письма сброса пароля (Reset Password) в ...