Skip to content

Commit

Permalink
Issue #605: remove _DisableBannerCheck()
Browse files Browse the repository at this point in the history
  • Loading branch information
bschmalhofer committed Nov 2, 2020
1 parent febe666 commit 3824710
Showing 1 changed file with 0 additions and 23 deletions.
23 changes: 0 additions & 23 deletions Kernel/Output/HTML/Layout.pm
Original file line number Diff line number Diff line change
Expand Up @@ -884,9 +884,6 @@ sub Login {
Data => \%Param,
);

# remove the version tag from the header if configured
$Self->_DisableBannerCheck( OutputRef => \$Output );

return $Output;
}

Expand Down Expand Up @@ -1609,9 +1606,6 @@ sub Header {
Data => \%Param
);

# remove the version tag from the header if configured
$Self->_DisableBannerCheck( OutputRef => \$Output );

return $Output;
}

Expand Down Expand Up @@ -4281,9 +4275,6 @@ sub CustomerHeader {
Data => \%Param,
);

# remove the version tag from the header if configured
$Self->_DisableBannerCheck( OutputRef => \$Output );

return $Output;
}

Expand Down Expand Up @@ -6023,20 +6014,6 @@ sub _BuildSelectionOutput {
return $String;
}

sub _DisableBannerCheck {
my ( $Self, %Param ) = @_;

return 1 if !$Kernel::OM->Get('Kernel::Config')->Get('Secure::DisableBanner');
return if !$Param{OutputRef};

# remove the version tag from the header
${ $Param{OutputRef} } =~ s{
^ X-Powered-By: .+? Open \s Ticket \s Request \s System \s \(http .+? \)$ \n
}{}smx;

return 1;
}

=head2 _RemoveScriptTags()
This function will remove the surrounding <script> tags of a
Expand Down

0 comments on commit 3824710

Please sign in to comment.