Skip to content

Commit

Permalink
Updated protocol to https for a secured communitation with gravatar.com.
Browse files Browse the repository at this point in the history
  • Loading branch information
NiklasSchmitt authored and dennykorsukewitz committed Aug 11, 2023
1 parent fcfcd8f commit 3a37544
Show file tree
Hide file tree
Showing 5 changed files with 7 additions and 6 deletions.
1 change: 1 addition & 0 deletions CHANGES.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,5 @@
# 6.5.4 2023-??-??
- 2023-08-11 Updated protocol to https for a secured communitation with gravatar.com.
- 2023-08-11 Fixed SelectAll function in AdminDynamicFieldScreenConfiguration. Select only all visible elements.
- 2023-08-11 Added additional note to DynamicFields::ScreenConfiguration::ConfigKeysOfScreensByObjectType###Framework for AgentTicketSearch (DynamicField|DefaultColumns).
- 2023-08-11 Fixed - Installing a package with dependency not possible - Added additional check if any problem exists - means if any package or module is not installed.
Expand Down
2 changes: 1 addition & 1 deletion Kernel/Output/HTML/Layout.pm
Original file line number Diff line number Diff line change
Expand Up @@ -1439,7 +1439,7 @@ sub Header {
my $DefaultIcon = $ConfigObject->Get('Frontend::Gravatar::DefaultImage') || 'mp';
$Kernel::OM->Get('Kernel::System::Encode')->EncodeOutput( \$Self->{UserEmail} );
$Param{Avatar}
= '//www.gravatar.com/avatar/' . md5_hex( lc $Self->{UserEmail} ) . '?s=100&d=' . $DefaultIcon;
= 'https://www.gravatar.com/avatar/' . md5_hex( lc $Self->{UserEmail} ) . '?s=100&d=' . $DefaultIcon;
}
else {
my %User = $Kernel::OM->Get('Kernel::System::User')->GetUserData(
Expand Down
2 changes: 1 addition & 1 deletion Kernel/Output/HTML/Preferences/Avatar.pm
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ sub Param {

if ( $AvatarEngine eq 'Gravatar' && $Self->{UserEmail} ) {
$Kernel::OM->Get('Kernel::System::Encode')->EncodeOutput( \$Self->{UserEmail} );
$Return->{Avatar} = '//www.gravatar.com/avatar/' . md5_hex( lc $Self->{UserEmail} ) . '?s=45&d=mp';
$Return->{Avatar} = 'https://www.gravatar.com/avatar/' . md5_hex( lc $Self->{UserEmail} ) . '?s=45&d=mp';
}

$Return->{UserEmailAddress} = $Param{UserData}->{UserEmail};
Expand Down
4 changes: 2 additions & 2 deletions Kernel/Output/HTML/TicketZoom/Agent/Base.pm
Original file line number Diff line number Diff line change
Expand Up @@ -185,7 +185,7 @@ Get URL used for article sender image.
Returns:
$SenderImage = '//gravatar.com/avatar/28a58af1db24962e81212115e7cac685?s=80';
$SenderImage = 'https://gravatar.com/avatar/28a58af1db24962e81212115e7cac685?s=80';
=cut

Expand Down Expand Up @@ -218,7 +218,7 @@ sub _ArticleSenderImage {
}
}
$Kernel::OM->Get('Kernel::System::Encode')->EncodeOutput( \$Email );
$Result = '//www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=' . $Size . '&d=' . $DefaultIcon;
$Result = 'https://www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=' . $Size . '&d=' . $DefaultIcon;
}
}

Expand Down
4 changes: 2 additions & 2 deletions scripts/test/Ticket/MetaArticle.t
Original file line number Diff line number Diff line change
Expand Up @@ -379,7 +379,7 @@ my $SenderImage = $Kernel::OM->Get('Kernel::Output::HTML::TicketZoom::Agent::Bas

$Self->Is(
$SenderImage,
'//www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=80&d=' . $DefaultImage,
'https://www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=80&d=' . $DefaultImage,
'Avatar link is generated successfully'
);

Expand All @@ -391,7 +391,7 @@ $SenderImage = $Kernel::OM->Get('Kernel::Output::HTML::TicketZoom::Agent::Base')
$Kernel::OM->Get('Kernel::System::Encode')->EncodeOutput( \$Email );
$Self->Is(
$SenderImage,
'//www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=80&d=' . $DefaultImage,
'https://www.gravatar.com/avatar/' . md5_hex( lc $Email ) . '?s=80&d=' . $DefaultImage,
'Avatar link is generated successfully with utf-8 chars.'
);

Expand Down

0 comments on commit 3a37544

Please sign in to comment.