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

Remove middleware: connect.bodyParser() #27

Merged
merged 1 commit into from
Dec 12, 2013
Merged

Remove middleware: connect.bodyParser() #27

merged 1 commit into from
Dec 12, 2013

Conversation

sbruchmann
Copy link

Because connect.bodyParser() is deprecated
and will be removed in Connect 3, this
commit removes it from the stack and
uses express.urlencoded() and
express.json() instead.

Visit the Connect Wiki for more information.

Because `connect.bodyParser()` is deprecated
and will be removed in Connect 3, this
commit removes it from the stack and
uses `express.urlencoded()` and
`express.json()` instead.

Visit the [Connect Wiki][wiki] for more information.

[wiki]: https://github.com/senchalabs/connect/wiki/Connect-3.0
@daffl
Copy link
Member

daffl commented Dec 12, 2013

Ah that's where that warning came from. Thanks!

daffl added a commit that referenced this pull request Dec 12, 2013
@daffl daffl merged commit 216954c into feathersjs:master Dec 12, 2013
@sbruchmann sbruchmann deleted the sbruchmann/remove-bodyParser branch December 12, 2013 04:28
@ekryski
Copy link
Contributor

ekryski commented Dec 12, 2013

💥 nice! Thanks for that @sbruchmann. That has been bugging me for a while.

daffl added a commit that referenced this pull request Aug 22, 2018
daffl pushed a commit that referenced this pull request Aug 23, 2018
daffl pushed a commit that referenced this pull request Aug 23, 2018
daffl added a commit that referenced this pull request Aug 23, 2018
daffl pushed a commit that referenced this pull request Aug 28, 2018
throw error if service.id is missing
daffl pushed a commit that referenced this pull request Aug 29, 2018
throw error if service.id is missing
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.

3 participants