Use CANONICAL_URL for sitemap route

[janus-reith]

Signed-off-by: Janus Reith mail@janusreith.de

Resolves #689
Impact: minor
Type: bugfix

Issue

Sitemap routes currently show undefined:// as req.protocol is not available in api routes.

Solution

As detailed in #689, this will use CANONICAL_URL and pass it to the sitemap query instead of parsing the url from the incoming request.

Breaking changes

None - I consider it unlikely that shops request sitemaps from a host name that differs from the one defined in CANONICAL_URL. If that is the case, these could adjust their sitemap api route individually.

Testing

1. Make sure your api provides a sitemap
2. Navigate to http://localhost:4000/sitemap.xml and make sure it returns proper urls in the loc field.

[willopez]

LGTM, thanks

[ghost]

Could this be backported and patched onto 3.1.0 since the 4.0 release may be a ways out?