[tools] testbot/cgi: In case of insufficient privileges redirect to the main page.
Francois Gouget
fgouget at codeweavers.com
Thu May 19 12:13:58 CDT 2022
There is no point asking already logged in users to log in again if they
don't have enough privileges.
Signed-off-by: Francois Gouget <fgouget at codeweavers.com>
---
testbot/lib/WineTestBot/CGI/PageBase.pm | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/testbot/lib/WineTestBot/CGI/PageBase.pm b/testbot/lib/WineTestBot/CGI/PageBase.pm
index 3a78a44d4..d68b97289 100644
--- a/testbot/lib/WineTestBot/CGI/PageBase.pm
+++ b/testbot/lib/WineTestBot/CGI/PageBase.pm
@@ -88,12 +88,15 @@ sub new($$$$@)
{
$self->CheckSecurePage();
my $Session = $self->GetCurrentSession();
- if (! defined($Session) ||
- ! $Session->User->HasRole($RequiredRole))
+ if (!defined $Session)
{
my $LoginURL = "/Login.pl?Target=" . uri_escape($ENV{"REQUEST_URI"});
exit($self->Redirect(MakeSecureURL($LoginURL)));
}
+ elsif (!$Session->User->HasRole($RequiredRole))
+ {
+ exit($self->Redirect("/"));
+ }
}
$self->_initialize(@_);
--
2.30.2
More information about the wine-devel
mailing list