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.

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

[BUG] Feathers calls create method twice #75

Closed
sbruchmann opened this issue Jun 5, 2014 · 4 comments
Closed

[BUG] Feathers calls create method twice #75

sbruchmann opened this issue Jun 5, 2014 · 4 comments

Comments

@sbruchmann
Copy link

Running this example app with Feathers.js 0.4.0 and sending an URL encoded POST request via curl to /tasks prints the following to the console:

$ create task
$ Thu Jun 05 2014 06:07:34 GMT+0200 (CEST) 'About to push [object Object]'
$ create task
$ Thu Jun 05 2014 06:07:34 GMT+0200 (CEST) 'About to push [object Object]'

It looks like tasks.create gets called twice, or am I missing something?
@sbruchmann
Copy link
Author

I just updated to version 1.0.0-pre.1 by directly cloning this repository into node_modules and the bug disappeared. Closing.

@daffl
Copy link
Member

daffl commented Jun 5, 2014

Odd... I don't think anything changed in that aspect. Did that happen with other clients (I usually use the Postman Chrome plugin) as well?

@sbruchmann
Copy link
Author

Yes, I also used Postman and plain XMLHttpRequests. The bug was always reproducible.

daffl pushed a commit that referenced this issue Aug 21, 2018
@kernelwhisperer @daffl
Allow data to be an array (#75)
a172d78
daffl pushed a commit that referenced this issue Aug 21, 2018
@kernelwhisperer @daffl
Allow data to be an array (#75)
f6560ca
daffl added a commit that referenced this issue Aug 21, 2018
@daffl
Remove empty channels (#75)
f986cba
daffl pushed a commit that referenced this issue Aug 21, 2018
@greenkeeper @daffl
fix(package): update socket.io to version 2.0.0 (#75)
75a4c57
daffl added a commit that referenced this issue Aug 21, 2018
@daffl
Remove setMaxListeners (#75)
c57a924
daffl added a commit that referenced this issue Aug 22, 2018
@daffl
Remove empty channels (#75)
086f15e
daffl pushed a commit that referenced this issue Aug 22, 2018
@greenkeeper @daffl
fix(package): update socket.io to version 2.0.0 (#75)
d4a4b71
daffl added a commit that referenced this issue Aug 22, 2018
@daffl
Remove setMaxListeners (#75)
c71abaf
daffl added a commit that referenced this issue Aug 29, 2018
@daffl
Rename repository and use npm scope (#75)
e5b41c5 
* Rename repository and use npm scope

* Prepare prerelease
daffl pushed a commit that referenced this issue Aug 29, 2018
@ekryski @daffl
Merge pull request #75 from enten/master
6605e3e 
Fix toLowerCase hook
daffl added a commit that referenced this issue Aug 29, 2018
@daffl
Rename repository and use npm scope (#75)
f17858e 
* Rename repository and use npm scope

* Prepare prerelease
daffl pushed a commit that referenced this issue Aug 29, 2018
@ekryski @daffl
Merge pull request #75 from enten/master
8c3670e 
Fix toLowerCase hook
@lock
Copy link

lock bot commented Feb 8, 2019

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue with a link to this issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Feb 8, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@sbruchmann @daffl