diff --git a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Plugins/GrantSession.pm b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Plugins/GrantSession.pm index 9ab0cad3d..43fbc1dd9 100644 --- a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Plugins/GrantSession.pm +++ b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Plugins/GrantSession.pm @@ -45,7 +45,12 @@ sub run { } # Avoid display notification if AuthResult is not null - return PE_BADCREDENTIALS if $req->authResult > PE_OK; + if ( $req->authResult > PE_OK ) { + $self->logger->debug( + "Bad authentication, do not check grant session rules"); + $req->userData( {} ); + return PE_BADCREDENTIALS; + } foreach ( sort sortByComment keys %{ $self->rules } ) { $self->logger->debug( "Grant session condition -> " diff --git a/lemonldap-ng-portal/t/61-GrantSession.t b/lemonldap-ng-portal/t/61-GrantSession.t index af99283a2..84db1f5ed 100644 --- a/lemonldap-ng-portal/t/61-GrantSession.t +++ b/lemonldap-ng-portal/t/61-GrantSession.t @@ -54,8 +54,8 @@ ok( $res->[2]->[0] =~ /<\/span><\/div>/, 'dwho rejected with PE_BADCREDENTIALS' ) or print STDERR Dumper( $res->[2]->[0] ); count(1); -ok( $res->[2]->[0] =~ m%Go to portal%, - 'Found goToPortal button' ) +ok( $res->[2]->[0] =~ m%Connect%, + 'Found connect button' ) or print STDERR Dumper( $res->[2]->[0] ); count(1); @@ -125,8 +125,8 @@ ok( 'rtyler rejected with PE_BADCREDENTIALS' ) or print STDERR Dumper( $res->[2]->[0] ); count(1); -ok( $res->[2]->[0] =~ m%Go to portal%, - 'Found goToPortal button' ) +ok( $res->[2]->[0] =~ m%Connect%, + 'Found connect button' ) or print STDERR Dumper( $res->[2]->[0] ); count(1);