Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Исправить временные редиректы #1390

Open
Tracked by #1388
ashikov opened this issue Nov 8, 2022 · 6 comments
Open
Tracked by #1388

Исправить временные редиректы #1390

ashikov opened this issue Nov 8, 2022 · 6 comments
Labels

Comments

@ashikov
Copy link
Contributor

ashikov commented Nov 8, 2022

Нужно исключить на сайте временные редиректы. Сейчас 302 редирект происходит при переходе по ссылкам с /en/ (пример https://sicp.hexlet.io/en/pages/about). Нужно перенастроить, чтобы происходил 301 редирект

@vasilysmolin
Copy link
Contributor

Взял в работу.
За редиректы отвечает пакет \Mcamara\LaravelLocalization\Middleware\ , и там жестко зашит статус ответа
return new RedirectResponse($redirection, 302, ['Vary' => 'Accept-Language']);

Я правильно понял, чот нужно подумать как это переопределить?

@fey
Copy link
Collaborator

fey commented Nov 22, 2022

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

@acidmaksim
Copy link
Contributor

@vasilysmolin получается сделать или нужна помощь?

@vasilysmolin
Copy link
Contributor

@vasilysmolin получается сделать или нужна помощь?

@acidmaksim думаю отказать от библиотек вообще, не смог найти подходящие альтернативы. Сделать вот такой подход: https://laravel.demiart.ru/multi-language-routes-and-locales-with-auth/ - смотрю чтобы всё легло как надо. Если есть библиотеки на примете или другой подход в решении, буду рад за подсказку двигаться в нужном направлении.

@ashikov
Copy link
Contributor Author

ashikov commented Jan 1, 2023

@vasilysmolin как тут дела?

@vasilysmolin
Copy link
Contributor

vasilysmolin commented Jan 13, 2023

@ashikov задача еще актуальна? вижу что англиские урлы сделали без en и редиректов сейчас нет вроде.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: No status
Development

No branches or pull requests

4 participants