Remove the query string from the CleanURL

[smarthall]

We are using Azure Blob storage as a private Helm repository. As described here. This breaks in helm-operator because the SAS token makes the cache directory too long and it fails with the following message:

synchronization of release 'demo' in namespace 'default' failed: failed to prepare chart for release: chart unavailable: mkdir /tmp/v3/[LONG FILE NAME]: file name too long

This PR strips the query string from the repo url that is used for caching so that:

1. If the authentication token changes the same cache can still be used.
2. The directory name is shorter

[hiddeco]

LGTM, thank you very much! This should be fine to be included in the next patch release 🥇

Before I merge, can you please rebase your PR?

[smarthall]

You are welcome, thanks for merging it.

I've rebased onto the latest master.