From 3435d22040709509e4e88532881416b0acf13f76 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 20 Nov 2020 09:04:30 +0100 Subject: [PATCH] Bug 27061: Remove duplicate permission check in svc/mana/search There is a call to check_cookie_auth then another one to get_template_and_user Test plan: Use mana to create new report and new subscription. Confirm that you need the necessary permissions to use it Signed-off-by: Owen Leonard Signed-off-by: Julian Maurice Signed-off-by: Jonathan Druart (cherry picked from commit 915c5d13bea82d8ff76e2c76525aa70a1f22961e) Signed-off-by: Fridolin Somers --- svc/mana/search | 8 -------- 1 file changed, 8 deletions(-) diff --git a/svc/mana/search b/svc/mana/search index 84b356c738..5b7b43f7a7 100755 --- a/svc/mana/search +++ b/svc/mana/search @@ -29,14 +29,6 @@ use CGI; my $input = CGI->new; -my ( $auth_status, $sessionID ) = - check_cookie_auth( $input->cookie('CGISESSID'), - { serials => 'create_subscription' } ); - -if ( $auth_status ne "ok" ) { - exit 0; -} - my $templatename; if ($input->param( "resource" ) eq 'report') { $templatename = "mana/mana-report-search-result.tt"; -- 2.39.5