dani/lemonldap-ng
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Improve code (#1658)

Browse Source
This commit is contained in:
Christophe Maudoux 2019-02-26 18:58:44 +01:00
parent 212be23313
commit cc99cec73a
1 changed files with 11 additions and 14 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Plugins/CheckUser.pm
View File

@ -3,8 +3,6 @@ package Lemonldap::NG::Portal::Plugins::CheckUser;
use strict;
use Mouse;
use Lemonldap::NG::Portal::Main::Constants qw(
PE_CONFIRM
PE_OK
PE_TOKENEXPIRED
PE_USERNOTFOUND
PE_NOTOKEN
@ -43,15 +41,12 @@ sub init {
sub check {
my ( $self, $req ) = @_;
my ( $hdrs, $attrs, $array_attrs, $array_hdrs ) = ( {}, {}, [], [] );
my $msg = 'checkUser';
my $result = '';
my $auth = 0;
my ( $attrs, $array_attrs, $array_hdrs ) = ( {}, [], [] );
my $msg = my $auth = '';
# Check token
if ( $self->conf->{requireToken} ) {
my $token = $req->param('token');
my $msg = '';
unless ($token) {
$self->userLogger->warn('CheckUser try without token');
$msg = PE_NOTOKEN;
@ -86,6 +81,7 @@ sub check {
$msg = 'PE' . $req->{error};
$attrs = {};
}
else { $msg = 'checkUser' }
# Create an array of hashes for template loop
$self->logger->debug("Delete hidden attributes");
@ -109,9 +105,9 @@ sub check {
$auth = $self->_authorization( $req, $url );
$self->logger->debug(
"checkUser requested for user: $req->{user} and URL: $url");
$result = $auth ? "allowed" : "forbidden";
$auth = $auth ? "allowed" : "forbidden";
$self->userLogger->notice(
"checkUser -> $req->{user} is $result to access: $url");
"checkUser -> $req->{user} is $auth to access: $url");
# Return VirtualHost headers
$array_hdrs = $self->_headers( $req, $url );
@ -136,11 +132,12 @@ sub check {
$self->p->checkXSSAttack( 'URL', $url ) ? ""
: $url
),
ALLOWED => $result,
ALERTE_AUTH => ($result eq 'allowed' ? "alert-success" : "alert-danger"),
HEADERS => $array_hdrs,
ATTRIBUTES => $array_attrs,
TOKEN => $token,
ALLOWED => $auth,
ALERTE_AUTH =>
( $auth eq 'allowed' ? "alert-success" : "alert-danger" ),
HEADERS => $array_hdrs,
ATTRIBUTES => $array_attrs,
TOKEN => $token,
}
);
}