qs comma parsing behavior isn't documented properly

[tzelonpulse]

According to the documentation, the default value of comma option is false, but the real default is true

https://github.com/adonisjs/http-server/blob/2c2f0b26973e61f6f6875edd0cf1af2ef81dd067/src/define_config.ts#L53
https://github.com/adonisjs/http-server/blob/428774e7baa4738e7b84df3bfd4b982e3c5332cb/src/types/qs.ts#L45

I'm not sure which is true. But in my opinion, comma should be false.

[thetutlage]

The types docblock is incorrect. We have enabled it on purpose, as many applications rely on sending comma separated values. I will fix the docblock and also happy to accept a PR for the same :)

[tzelonpulse]

I know it is probably too late to change that decision.
The reason I noticed it is because when sending a query params search=ab, the parser parses it as an array and not a string. The only solution I can think for it is to encode the search value before sending it.
I use axios btw.

PS
I would be glad to send a pr with the doc change.

[thetutlage]

Yeah, changing the behavior will be a breaking change. Maybe you can update the config of your app and for now we can update the docs