Fixed Method getCoverImage may not return canonical URL in some cases

[truonglv]

Affected version

2.2.8

If the image URL is built proxy (XF service) it's not canonicalize.

 

[Jeremy P]

Sorry, I don't think I understand. The cover images seem to work as anticipated when the image URLs are proxied. Can you post the steps required to reproduce this issue?

 

[truonglv]

Sorry, I don't think I understand. The cover images seem to work as anticipated when the image URLs are proxied. Can you post the steps required to reproduce this issue?

In the entity XF\Entity\Thread line 723

$coverImageUrl = $this->app()->stringFormatter()->getProxiedUrlIfActive('image', $url);

It's return /proxy.php?.... while whole function return canonical URL. So I think it would be canonical as well.

 

[XF Bug Bot]

Thank you for reporting this issue, it has now been resolved. We are aiming to include any changes that have been made in a future XF release (2.2.9).

Change log:

Canonicalize proxied thread cover image URLs

There may be a delay before changes are rolled out to the XenForo Community.