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

404 url list and redirect creation #45

Open
alexander-schranz opened this issue Mar 2, 2016 · 6 comments
Open

404 url list and redirect creation #45

alexander-schranz opened this issue Mar 2, 2016 · 6 comments
Assignees

Comments

@alexander-schranz
Copy link
Member

Mostly when you create a new website you can not create for all old urls redirects.
It would be good to have a 404 url list in the backend foreach webspace with a counter (which urls was called most) and the content manager create a redirect to a specific page / url.

@danrot
Copy link
Contributor

danrot commented Mar 2, 2016

Sounds like a great idea! But we have to think about how to handle all the discovery requests like /phpmyadmin. We probably have to check if there is already a webspace configured for this URL, and it would probably also be a good idea to have a blacklist.

And when we finally release that we should not forget to put that into the kernel.terminate event, so that the response time doesn't get worse :-)

@QuingKhaos
Copy link

and the content manager create a redirect to a specific page / url.

As pages with internal url target? Or with a redirect settings module?

@webmozart
Copy link

Such an error log is indeed quite standard. The most useful ones I've seen let you either discard errors only once (in that case all errors of the same type are removed) or discard them forever (all errors of the same type are removed, future errors are not shown in the log).

@chirimoya chirimoya self-assigned this Mar 7, 2016
@alexander-schranz
Copy link
Member Author

Tracking and grouping 404 requests and make it easy to create a Redirect for this url would maybe something which could be implemented in the SuluRedirectBundle and an optional activateable feature /cc @wachterjohannes

@danrot
Copy link
Contributor

danrot commented Oct 16, 2019

@alexander-schranz Sounds reasonable, should we move that issue then?

@alexander-schranz
Copy link
Member Author

Yeah will move the issue to the redirectbundle

@alexander-schranz alexander-schranz transferred this issue from sulu/sulu Oct 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

5 participants