Thanks to visit codestin.com
Credit goes to github.com

Skip to content

rebase: Generic Body Parser implemented #544

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 26 commits into from
Closed

rebase: Generic Body Parser implemented #544

wants to merge 26 commits into from

Conversation

@ctcpip
Copy link
Member

@ctcpip ctcpip commented Oct 16, 2024

supersedes #524

inigomarquinez and others added 26 commits May 24, 2024 17:21
@inigomarquinez @ctcpip
chore: add support for OSSF scorecard reporting (expressjs#522)
9eb5ab4 
PR-URL: expressjs#522
@ctcpip
Added support for external parsers to bodyParser.json()
1c2f923
@ctcpip
removed test dependency on json-bigint
2f4913c
@ctcpip
reworked doc to describe json parser() func better
6daefb5
@ctcpip
added parser() option and doc for .text()
b740295
@ctcpip
added parser() option and doc for .raw()
cf3a8e3
@ctcpip
added parser() option and doc for .urlencoded()
2468f72
@ctcpip
cleanup to satisfy linter
d75adee
@ctcpip
added generic parser
8224cda
@ctcpip
converted json parser to use generic parser
a5adc82
@ctcpip
converted raw parser to use generic parser
c572330
@ctcpip
converted text parser to use generic parser
7390c89
@ctcpip
converted urlencoded parser to use generic parser
f4f9d84
@ctcpip
cleanup / fix linter warnings
10eb0af
@ctcpip
removed items from README
67060ca
@ctcpip
added bodyParser.generic() getter
e6f6004
@ctcpip
cleanup / fix linter warnings
07e0602
@sdellysse @ctcpip
fixed tests after rebase
bb7697a
@sdellysse @ctcpip
satisfying linter
6a7c24d
@sdellysse @ctcpip
Ref'd genParser via the bodyparser getter to signal how third party p…
bc7454b 
…arsers should import genParser'
@sdellysse @ctcpip
removed dep on object-assign, which didnt support node < 0.10
519f306
@sdellysse @ctcpip
minor text cleanup
cdf46f0
@ctcpip
🔧 add debug script
8ac04fd
@ctcpip
🐛 fix object merging
1a43910
@ctcpip
🔥 clean up
86804fe
@ctcpip
💚 remove node < 4 from CI
88f84bd
@ctcpip ctcpip mentioned this pull request Oct 16, 2024
Closed
@ctcpip
Copy link
Member Author

ctcpip commented Oct 16, 2024

and now this needs a rebase again! 😄

This was referenced Nov 14, 2024
Merged
Closed
@Phillip9587 Phillip9587 mentioned this pull request Nov 19, 2024
Open
@ctcpip ctcpip mentioned this pull request Jan 14, 2025
Closed
@ctcpip ctcpip closed this Jan 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ctcpip @inigomarquinez @sdellysse