Fragments are valid in oEmbed scheme URLs

Update the comment as well

I agree that fragments are valid in scheme URLs. The patch workflow is incompatible with GitLab CI, so we'll need an MR to proceed. Let's also add test coverage (Drupal\Tests\oembed_providers\Unit\UrlIsValidTest) for 1) disallowing user and pass elements and 2) allowing fragments.

@mfb Thanks!

Great, thanks! By the way, to actually make use of this, I also found this issue over in drupal core: 🐛 OEmbed generates URLs with URL-decoded query string Needs review

Automatically closed - issue fixed for 2 weeks with no activity.