feat: add support of objects in query params#1075
feat: add support of objects in query params#1075aleksej-axiology wants to merge 1 commit intoacacode:mainfrom
Conversation
|
herman-axiology
force-pushed
the
feat/objects-support-in-query-params
branch
from
91b106f to
a8851f7
Compare
|
bugbot run |
![cursor[bot]](https://avatars.githubusercontent.com/in/1210556?s=60&v=4){kind=small}
| .join("&"); | ||
| .map((key) => | ||
| typeof query[key] === "object" || Array.isArray(query[key]) | ||
| ? this.addArrayQueryParam(query, key) |
There was a problem hiding this comment.
Bug: Null Handling Error in Query String Method
The toQueryString method incorrectly treats null values as objects, causing addArrayQueryParam to receive null. This results in a TypeError when Object.entries(null) is called.
|
Thank you for looking at this @smorimoto 🙇♂️ Do you think this could get accepted? |
3 participants
Note
Enhances query serialization to handle objects and arrays using bracketed keys (key[index]/key[subkey]) in
fetch-http-client.ejs.templates/base/http-clients/fetch-http-client.ejs):addArrayQueryParam: now serializes arrays askey[index]=valueand objects viaObject.entries, emittingkey[subkey]=value.toQueryString: treats both objects and arrays as complex params, delegating toaddArrayQueryParam; primitives useaddQueryParam.Written by Cursor Bugbot for commit a8851f7. This will update automatically on new commits. Configure here.