Bug: build_url may remove trailing slash

[relikd]

Found this while browsing through the source. Havent tested whether this is actually an issue, but:

lektor/lektor/utils.py

Lines 666 to 668 in dbae91e

	_, last = url.split("/", 1)
	if "." in last:
	return url

shouldn't that actually be

last = url.split("/")[-1]
# or:
# if "." in url.split("/")[-1]:

Otherwise, an URL like this /path.my/page/2/ may wrongly remove the trailing slash.

[dairiki]

I think you're right about the split.

It was probably meant to be

_, last = url.rsplit("/", 1)

Personally, I'd write it:

_, _, last = url.rpartition("/")

(Perhaps a larger refactor/cleanup is possible. Maybe using pathlib.PurePosixPath?)

As you say, it would be nice to have an actual test/demo of how it can go wrong.