fix: endpoints in dev server should never have trailing slash

[ematipico]

Changes

Closes #10149

Testing

Added a new test case. It seems we didn't have one.

Docs

[changeset-bot]

🦋 Changeset detected

Latest commit: 4be7253

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[lilnasy]

Assuming the changeset needs to be corrected, I think this is the wrong call.

Users should configure "trailingSlash" to inform the dev server that the production server will behave a certain way so it should too. If the behavior doesn't match, they may need to configure something else. trailingSlash is doing what the config is telling it to.

[matthewp]

Hm, that's a good point, let's chat about it.

[lilnasy]

Test cases already exist in test/units/routing/trailing-slash.test.js, and check for the opposite behavior.

[kripod]

I‘m afraid this issue still persists: #10149 (comment)

Static File Endpoints exposing e.g. .png assets for dynamic social image generation can‘t be accessed as /img.png in dev mode, because only /img.png/ would pass the overly strict trailing slash requirements.