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

0.0.28 body-parser prevents proxy POST requets #723

Closed
zigomir opened this Issue May 14, 2014 · 2 comments

Comments

Projects
None yet
3 participants
@zigomir
Contributor

zigomir commented May 14, 2014

This commit prevents my app to proxy a POST request to my api server.

When I remove app.use(bodyParser()); all works fine...

I'm using ember canary, ember-data beta.7 with ActiveModelAdapter.

@jzgit

This comment has been minimized.

jzgit commented May 14, 2014

same here - thanks for the fix @zigomir!

@rwjblue

This comment has been minimized.

Contributor

rwjblue commented Jun 15, 2014

The server stuff is no longer automatically generated. You must use ember g api-stub to generate one now.

Closing, as a new application out of the box should work properly now.

@rwjblue rwjblue closed this Jun 15, 2014

homu added a commit that referenced this issue Oct 23, 2016

Auto merge of #6377 - ember-cli:greenkeeper-nock-9.0.0, r=kellyselden
Update nock to version 9.0.0 🚀

Hello lovely humans,

[nock](https://www.npmjs.com/package/nock) just published its new version 9.0.0.

<table>
  <tr>
    <th align=left>
      State
    </th>
    <td>
      Update 🚀
    </td>
  </tr>
  <tr>
    <th align=left>
      Dependency
    </td>
    <td>
      nock
    </td>
  </tr>
  <tr>
    <th align=left>
      New version
    </td>
    <td>
      9.0.0
    </td>
  </tr>
  <tr>
    <th align=left>
      Type
    </td>
    <td>
      devDependency
    </td>
  </tr>
</table>

This version is **not covered** by your **current version range**.

Without accepting this pull request your project will work just like it did before. There might be a bunch of new features, fixes and perf improvements that the maintainers worked on for you though.

I recommend you look into these changes and try to get onto the latest version of nock.
Given that you have a decent test suite, a passing build is a strong indicator that you can take advantage of these changes by merging the proposed change into your project. Otherwise this branch is a great starting point for you to work on the update.

Do you have any ideas how I could improve these pull requests? Did I report anything you think isn’t right?
Are you unsure about how things are supposed to work?

There is a collection of [frequently asked questions](https://greenkeeper.io/faq.html) and while I’m just a bot, there is a group of people who are happy to teach me new things. [Let them know](https://github.com/greenkeeperio/greenkeeper/issues/new).

Good luck with your project 

You rock!

🌴

---
The new version differs by 51 commits .

- [`b3caf8a`](nock/nock@b3caf8a) <code>9.0.0</code>
- [`2f69b19`](nock/nock@2f69b19) <code>Drop support for node versions 0.10, 0.11, 0.12 and 5</code>
- [`1f7a691`](nock/nock@1f7a691) <code>Add section on node versions support</code>
- [`2cf2427`](nock/nock@2cf2427) <code>Update CHANGELOG</code>
- [`540720d`](nock/nock@540720d) <code>8.2.0</code>
- [`0712456`](nock/nock@0712456) <code>Add changelog to the dev dependencies</code>
- [`2ecf1ec`](nock/nock@2ecf1ec) <code>Ignore browserify-bundle.js</code>
- [`28acae0`](nock/nock@28acae0) <code>Update browserify-bundle.js</code>
- [`d5962f2`](nock/nock@d5962f2) <code>Merge pull request #723 from pimterry/optional-mocks</code>
- [`142c818`](nock/nock@142c818) <code>Include all review markup changes in browserify-bundle</code>
- [`e72ab0a`](nock/nock@e72ab0a) <code>Add .activeMocks() to recreate previous .pendingMocks() behaviour</code>
- [`475d605`](nock/nock@475d605) <code>Add times+optional and persist+optional tests</code>
- [`475c7a5`](nock/nock@475c7a5) <code>Document assumption in pendingMocks()</code>
- [`143e42e`](nock/nock@143e42e) <code>Clarify optional mock behaviour a little in README</code>
- [`5af77f0`](nock/nock@5af77f0) <code>Merge pull request #721 from pimterry/removeInterceptor-from-pending</code>

There are 51 commits in total. See the [full diff](nock/nock@c7b156a...b3caf8a).

---
This pull request was created by [greenkeeper.io](https://greenkeeper.io/).

<sub>Tired of seeing this sponsor message? ⚡️ `greenkeeper upgrade`</sub>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment