Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: mod fastify querystring parser #355

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

casheeeewnuts
Copy link

Types of changes

Changes

  • add library 'qs' and '@types/qs' in server/pretense/package.json
  • configure fastify instance with querystringParser: qs.parse in entry point modules

Additional context

Community note

Please upvote with reacting as 👍 to express your agreement.

@casheeeewnuts casheeeewnuts changed the title Mod fastify query parser fix: mod fastify querystring parser Apr 14, 2022
@LumaKernel
Copy link
Contributor

なるほど、qsを使用しておかないとaspidaが送るものとの整合性が取れないのか。
@aspida/axios はおそらくそうなのだろうが @aspida/fetch がどうなのかは確認が必要だな。その他の@aspida/*で違う挙動を取る場合は、それはそれで別にやらなければいけないが、axiosのときに難なく動く状態のこのPRをマージするのはひとまず良さそう。

Copy link
Contributor

@LumaKernel LumaKernel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ありがとうございます。

@LumaKernel LumaKernel self-requested a review April 18, 2022 09:47
@LumaKernel
Copy link
Contributor

aspida/aspida#748
上記と似ていますね。

以下、メンバーと話して更に検討したうえでの現在の考えです。

@aspida/axios と frourio-express が本来その責任範囲外であるような [] 付加・取り外しをする、axios/expressデフォルト動作を抑えていないのが修正されるべきで、querystring同等の動作をさせたい場合は個別のプロジェクトで対処するのが良いではないかというふうに考えています。

query: {id: string[]}id[]=... が勝手に飛んでいた @aspida/axios を修正して、もしほしいなら、 query: { 'id[]' : string[]} と書かなければいけないようにするほうが良いと思っています。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

cannot access to array query parameter with defined parameter name when axios and fastify
2 participants