Swap qs for neoqs

[mattstrayer]

Followup from -> adonisjs/bodyparser#65

🔗 Linked issue

❓ Type of change

• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality like performance)
• ✨ New feature (a non-breaking change that adds functionality)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[PuruVJ]

Just in case lucid ships with CJS, this should be neoqs/legacy

[RomainLanz]

Thanks for your PR!

We discussed this internally, and we would prefer not to migrate to neoqs as of now. But we keep the PR open for further discussion.

The main reason is that qs is a critical part of AdonisJS, and this new package may not fit all use cases or keep up with the qs package.

Also, migrating to a brand new package may add more vulnerability since we don't know well the author nor its future intention.

If we want to debloat qs, we would prefer to make our own version under the BoringNode organization.

Related PRs are:

• Replace qs with zero-dependency & Typesafe neoqs bodyparser#65
• Replace qs with neoqs http-server#88
• Swap qs for neoqs #1048