Project

General

Profile

Bug #5454

2.3 alpha - system_advanced_admin.php - webgui redirect/reload test doesn't work, ignores port config unchanged

Added by Stilez y almost 4 years ago. Updated almost 4 years ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
11/15/2015
Due date:
% Done:

0%

Estimated time:
Affected Version:
2.3
Affected Architecture:

Description

The page should only redirect when the IP, port or SSL status has changed, but it always redirects even when they don't.

Test:

Go to the page.
Change something trivial (I changed the "display page name first" setting)
save

result - "One moment...redirecting to https://IP:PORT/system_advanced_admin.php in 20 seconds" even though the webgui URI hasn't changed and the code should only result in a redirect action of this kind when it changes?

History

#1 Updated by Steve Beaver almost 4 years ago

  • Status changed from New to Confirmed
  • Assignee set to Steve Beaver
  • Target version set to 2.3
  • Affected Version set to 2.3

I observed the same behavior last week before adding the "Web GUI CSS" selector and added it to my list. I should be able to get to it in the next couple of days.

#2 Updated by Stilez y almost 4 years ago

At the same time, maybe add a direct link "...or click here to immediately reload..." (or the link itself made a hyperlink) so the 20 second pause can be shortcutted?

#3 Updated by Steve Beaver almost 4 years ago

  • Status changed from Confirmed to Resolved

Code around: $_POST['webgui-redirect'] restarted the webgui if the value was 'yes', or if it was not :) Changed it to restart only if the value has changed WRT to the current value.

Also available in: Atom PDF