normalize_path_middleware fix not backported (#3670)

[dmorgoon]

I see that fix introduced in #3670 is still not backported to current version (at the moment of writing - aiohttp 3.14.5).
I would to know why, for what reasons? Are there any bugs in that fix?

[Pixeless001]

Doesn't look like there was a bug in it. #3670 was merged into master back in 2019 and just never got a backport label or a 3.x PR, so it only ever landed on master. Master still has the fix (the path.endswith("/") check) and the test_bug_3669 test, so it wasn't reverted either.

3.14.5 and the 3.15 branch still have the old line:

if merge_slashes and remove_slash:

I ran the test from #3670 against 3.14.5 and it still fails: with remove_slash=True, GET /paymethods gets a 308 to /paymethod instead of a 404.

So I think it was just missed. It's a one-line fix plus the test, so backporting it to 3.15 should be pretty easy if the maintainers want it.

[dmorgoon]

@Pixeless001 thanks for your PR

[Dreamsorcerer]

It predates the backport CI check, so was among hundreds that got missed by the looks of it. Feel free to open a backport following the instructions: #3670 (comment)