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 cannot process array of values in application/x-www-form-urlencoded request #1873

Merged
merged 3 commits into from Apr 24, 2022

Conversation

ChandanChainani
Copy link
Contributor

@ChandanChainani ChandanChainani commented Apr 22, 2022

Fix #1872

@welcome
Copy link

welcome bot commented Apr 22, 2022

Thanks for opening this pull request! 🎉 Please check out our contributing guidelines. If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@ChandanChainani ChandanChainani changed the title Fix form square brackets Fix cannot process array of values in application/x-www-form-urlencoded request Apr 22, 2022
@ReneWerner87
Copy link
Member

ReneWerner87 commented Apr 22, 2022

Thx, can you add an additional test case for this.

@ChandanChainani
Copy link
Contributor Author

ChandanChainani commented Apr 22, 2022

@ReneWerner87 Added test cases please review.

@ReneWerner87 ReneWerner87 merged commit 941ada9 into gofiber:master Apr 24, 2022
16 checks passed
@welcome
Copy link

welcome bot commented Apr 24, 2022

Congrats on merging your first pull request! 🎉 We here at Fiber are proud of you! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

trim21 pushed a commit to trim21/fiber that referenced this issue Aug 15, 2022
…ded` request (gofiber#1873)

* Renamed parseQuery to parseParamSquareBrackets

* Added square brackets processing for `application/x-www-form-urlencoded`

* Added test cases for Array of form values
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants