@Param only accepts numbers

I'm trying to use a route like this: @delete('/account/:apiKey')

The apiKeys are strings and not numbers. If I try to hit the endpoint with a string I get a bad request (500) error:

Error
at ParameterParseJsonError.HttpError [as constructor] (/myProjectDirectory/src/http-error/HttpError.ts:19:22)
at ParameterParseJsonError.BadRequestError [as constructor] (/myProjectDirectory/src/http-error/BadRequestError.ts:10:9)
at new ParameterParseJsonError (/myProjectDirectory/src/error/ParameterParseJsonError.ts:10:9)
at ActionParameterHandler.parseValue (/myProjectDirectory/src/ActionParameterHandler.ts:153:23)
at ActionParameterHandler.normalizeParamValue (/myProjectDirectory/src/ActionParameterHandler.ts:137:34)
at ActionParameterHandler.handle (/myProjectDirectory/src/ActionParameterHandler.ts:44:28)
at /myProjectDirectory/src/RoutingControllers.ts:116:49
at Array.map ()
at RoutingControllers.executeAction (/myProjectDirectory/src/RoutingControllers.ts:116:14)
at /myProjectDirectory/src/RoutingControllers.ts:83:33

If I use a number, I get my own defined 404 custom error as expected.

Wait, I realized that it can actually work with "string" but not "String".
I changed it from:
deleteAccount(@res() response: any, @param("apiKey") apiKey: String) {
to:
deleteAccount(@res() response: any, @param("apiKey") apiKey: string) {

and it worked.

[MichalLytek]

It's weird but I'm glad you figured out how to fix it 😉

[NoNameProvided]

Hi @buzzyboy!

This issue happened because the String type matches strings created via the String constructor, aka new String('your-api-key'). You should never set any Typescript type to String, Number or Boolean. It's considered a bad practice and listed in the official "Typescript Dont's" list. You always should use the lower-case variants.

[github-actions]

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