Remove checking base_url in TrustedHosts check

So what was this postponed for?

I think banviktor was saying that the core api was changing, but that's no longer true so it seems fine to make this back to needs review or needs work.

Rerolled

bumping this one for the refractor of trusted hosts

I noticed this changes the token from @file to !file:

-    $paragraphs[] = $this->t('If the site should be available only at that URL it is recommended that you set it as the $base_url variable in the settings.php file at @file.', ['@file' => $settings_php]);
-    $paragraphs[] = $this->t('If the site has multiple URLs it can respond from you should whitelist host patterns with trusted_host_patterns in settings.php.');
+    $paragraphs[] = $this->t('If the site has multiple URLs it can respond from you should whitelist host patterns with trusted_host_patterns in settings.php at !file.', ['!file' => $settings_php]);

Is that an intentional and required change?

I couldn't explain why but !file: was causing an error and not being replaced.

Other then that though does this change still work for you also?

I think (?) that's a really important thing to figure out to be sure it's not a pathway for xss.

Can try to find some php doc but not sure where to look quickly, this is a current bug. It throws an error currently not replacing

@smustgrave What's the exact error? It could be that core is seeing it as a URL and thus a potential security issue when used as a replacement of the wrong type.

Actually just checked in D10 and no longer seeing the issue will update the patch later.

Okay this should be good to go!

Going ahead and merging to keep the ball rolling on the trusted host updates.