Skip to content

Remove middleware: connect.bodyParser()#27

Merged
daffl merged 1 commit into
feathersjs:masterfrom
sbruchmann:sbruchmann/remove-bodyParser
Dec 12, 2013
Merged

Remove middleware: connect.bodyParser()#27
daffl merged 1 commit into
feathersjs:masterfrom
sbruchmann:sbruchmann/remove-bodyParser

Conversation

@sbruchmann

Copy link
Copy Markdown

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

daffl commented Dec 12, 2013

Copy link
Copy Markdown
Member

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

ekryski commented Dec 12, 2013

Copy link
Copy Markdown
Contributor

💥 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