By default redirect to post index page after login

[yceruto]

This PR change the default target path / to admin_index blog_index route, useful when we access to login page directly (i.e. through URL).

[bocharsky-bw]

Looks like in this case, users, which don't have access to the admin page, will have an "Access denied" error.

[yceruto]

@bocharsky-bw you're right, but actually already happening, at least the most of times, the users accessing to login page through "browser backend" button and they get a "Access Denied" (and it makes sense if it was your intention).

Something like "redirect to post index for users" or "redirect to admin index for admins" would be the best approach.

Any suggestion to improving it?

[stof]

@yceruto if you want to do it, you need to create a dedicated route (used as target path) for which the controller would do redirections conditionally.

[javiereguiluz]

I'm divided about this proposal. If we do this right, we must create a listener to redirect to the best page depending on each user ... but that may be a bit overkill for this demo app.

Should we go all in for the listener ... or abandon this idea? Thanks!

[yceruto]

Something like "redirect to post index for users" or "redirect to admin index for admins" would be the best approach.

I'm agree that @stof's and @javiereguiluz's solution may be a bit overkill for this demo app and I abandon this idea.

But, my initial goal here was to find a better default_target_path that / for "the user" (admin or not) after login "only in this case:"

When the user access to login page directly (i.e. through URL) then this option (default_target_path) is used and not through "Browse Backend" button from home page, which use the target path saved in session.

So, to make it simpler and to avoid "Access denied" for users, I've changed again the default_target_path option to blog_index instead.

What do you think now?

[javiereguiluz]

👍