Canoncial URL should use Url::current() instead of Url::full()

[chescos]

When setting the config option meta.defaults.canonical to null, the canonical URL will be set from the value of Url::full() instead of Url::current(). Url::full() includes query parameters and the canonical URL is usually used to strip away query parameters like UTM tags, search parameters, and so on.

I mean, what's even the point of specifying a canonical URL from the full URL? That's exactly the same result as not setting a canonical URL at all.

[robinparisi]

Indeed, the current use of Url::full() bring no benefit for SEO since it will just use the full URL and generate a lot of duplicate content.

Expected behaviour (current page => canonical URL):

https://example.com?utm_source=email => https://example.com
https://example.com/products?filter=asc => https://example.com/products
https://example.com/page?I_will_harm_your_SEO => https://example.com/page

This is the best default behavior because all important parameters for SEO should preferably use URL rewriting (so compatible with Url::current())

[J-Brk]

XSS #247

[Barcelonczyk]

I totally agree that using Url::full() has no benefit for SEO.

It's worth mentioning that the config file is inconsistent in the current situation:

seotools/src/resources/config/seotools.php

Line 17 in ca27cd0

'canonical' => false, // Set null for using Url::current(), set false to total remove

seotools/src/resources/config/seotools.php

Line 41 in ca27cd0

'url' => false, // Set null for using Url::current(), set false to total remove

Based on the config file both opengraph url and canonical should use Url::current() but in fact, only opengraph url does it.

So using the same config for opengraph url and canonical we will receive e.g.

[vinicius73]

Fixed on #255