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

[Bug]: setting login.php as frontpage ends in too many redirection #5335

Open
Jimmi08 opened this issue Sep 4, 2024 · 0 comments
Open

[Bug]: setting login.php as frontpage ends in too many redirection #5335

Jimmi08 opened this issue Sep 4, 2024 · 0 comments
Labels
type: bug A problem that should not be happening

Comments

@Jimmi08
Copy link
Contributor

Jimmi08 commented Sep 4, 2024

What e107 version are you using?

v2.3.3

Bug description

When you set Frontpage in the admin area to login.php, the site ends in too many redirections.

In index,php is $inc set to login.php
then this file is included include($inc);
e_SELF is "" {if you don't use index.php)
e_LOGIN is login.php
then this test is true: e_LOGIN != e_SELF in login.php and this code is executed:

$prev = e107::getRedirect()->getPreviousUrl();
	if(!empty($prev))
	{
		e107::redirect($prev);
		exit();
	}
	e107::redirect();
	exit();

the previous URL is the homepage and this homepage again redirects to login.php
and site loading fails

How to reproduce

set frontpage as login.php
image

Expected behavior

working site?

What browser(s) are you seeing the problem on?

Chrome / Brave

PHP Version

8.1

@Jimmi08 Jimmi08 added the type: bug A problem that should not be happening label Sep 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A problem that should not be happening
Projects
None yet
Development

No branches or pull requests

1 participant