yarl update broke static file urls in templates

[14droplets]

With yarl version 1.9.2 static files urls with url_for(filename="") as used, for example, here:

aiohttp-debugtoolbar/aiohttp_debugtoolbar/tbtools/tbtools.py

Line 210 in 6570cb3

static_path = request.app.router[STATIC_ROUTE_NAME].url_for(filename="")

are no longer works correctly. Produced URL no longer contains terminating slash to be concatenated with filename in template code like it is done here:

aiohttp-debugtoolbar/aiohttp_debugtoolbar/templates/toolbar.jinja2

Line 10 in 6570cb3

<link rel="stylesheet" href="{{ static_path }}css/debugger.css" type="text/css">

I traced issue to this line in yarl: https://github.com/aio-libs/yarl/blob/1f94e1a7833a27203d63f8618d01d4d4fa2d3520/yarl/_url.py#L719. With segment as empty string - main loop breaks immediately.

Issue is observed with yarl versions 1.9.2 and 1.9.1 and disappears with yarl version 1.8.2.

I can rewrite templates using url_for instead of {{ static_path }}css/debugger.css like syntax and provide a pull request. Is it acceptable fix? Should I do it?

[Dreamsorcerer]

aiohttp-jinja2 has static(): https://aiohttp-jinja2.readthedocs.io/en/stable/#default-globals

So, if you can update to that, that would be perfect.

[14droplets]

Thank you, interesting feature. But using that would require to mess with app['static_root_url'] which may be used by user.
I will try to do something like this but autonomous

[Dreamsorcerer]

Oh, yes, we can't use that until #71 is done.
Will need to stick with a hack for now.

[14droplets]

Hacky hack goes brrr. As hacky as I could, but without ANY additional pollution to globals