Incorrect URLs of Drupal theme CSS and JS

Can you confirm you followed the configuration guide for Storybook?

https://git.drupalcode.org/project/cl_server/-/blob/1.x/docs/storybook.md

Confirming that we are seeing something similar:

https://foo.com/public:/css/css_zxa1V-vXt55-TF0hf24Omulyjsuhl19p9MPKhITP...

Difference is that for us, there is only one `/` after `public:` whereas poster has two.

But a similarity I can't help but note is that we are both using css aggregation.

  if ($route_name !== 'cl_server.render') {
    return;
  }
  $uri = implode('/', [$request->getSchemeAndHttpHost(), ltrim($uri, '/')]);

I wonder how aggregation changes the $route_name.

As a side note, and as a matter of slightly more readable code, I might patch as follows, but it won't solve the bug.

  if ($route_name === 'cl_server.render') {
    $uri = implode('/', [$request->getSchemeAndHttpHost(), ltrim($uri, '/')]);
  }

We just ran into this one too. It can be worked around by disabling CSS and JS aggregation. However, if we check for the scheme, \Drupal\Core\File\FileUrlGenerator::doGenerateString() will make this absolute and the correct hostname will be used.

I've opened an MR fixing this.

This is a good catch! I didn't anticipate the need to have aggregation on while developing with CL Server on. But I see it's a fair ask.

Re-opeining to port to 2.x as well.

Thanks for the quick work on this @deviantintegral & @e0ipso. I'm looking at the fix, and thinking about... what happens to `private://`

It seems what we need, rather than string matching the name of the stream wrapper, a solution that considers the kind of stream wrapper we are dealing with and adjusts accordingly.

We bear in mind too, that public:// is just a convention within Drupal.

I still think the fix is a step in the right direction. I'm just not sure it is complete. I guess we'll see. In any case. thank you for the fix.