Fix: Remove trailing slash from route path in KoaDriver

[Aareksio]

Description

Before the update, @koa/router allowed path declared with a trailing slash to be accessible both with or without trailing slash

router.get('/user/', handler)

// GET /user -> 200
// GET /user/ -> 200

After the update, only trailing slash path is accessible. This makes registering nested routes with @Get('/') infeasible, when one does not use trailing slashes for URLs.

@JsonController('/user')
export class UserController {
  @Get('/') // <-- this causes the route to become `/user/`
  getUser() {}
}

Proposed change, restricted in scope to affected KoaDriver only, removes the trailing slash if one appears in the route (aside from /). @koa/router allows such paths to be accessed both with and without trailing slash, as before.

Please note @Get('/') is what official tests are using, and it works just fine with express:
https://github.com/typestack/routing-controllers/blob/develop/test/functional/controller-base-routes.spec.ts#L18-L21

Checklist

• the pull request title describes what this PR does (not a vague title like Update index.md)
• the pull request targets the default branch of the repository (develop)
• the code follows the established code style of the repository
  • npm run prettier:check passes
  • npm run lint:check passes
• tests are added for the changes I made (if any source code was modified)
• documentation added or updated
• I have run the project locally and verified that there are no errors

Fixes

Fixes #1113

[attilaorosz]

This is a nice catch, thanks for the contribution!

[github-actions]

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.