Skip to content
This repository has been archived by the owner on Mar 22, 2022. It is now read-only.

restrictToOwner does not support multi patch, update and remove #228

Closed
daffl opened this issue Jul 1, 2016 · 2 comments
Closed

restrictToOwner does not support multi patch, update and remove #228

daffl opened this issue Jul 1, 2016 · 2 comments
Labels
Bug

Comments

@daffl
Copy link
Member

daffl commented Jul 1, 2016

It also throws a weird error in https://github.com/feathersjs/feathers-authentication/blob/master/src/hooks/restrict-to-owner.js#L15 due to id being null.
@ekryski
Copy link
Member

ekryski commented Jul 7, 2016

Ya we should be checking the method instead of assuming the id is present.

@ekryski ekryski added the Bug label Jul 7, 2016
ekryski added a commit that referenced this issue Aug 10, 2016
@ekryski
fix restrict to owner hook methods. Closes #228
2c49e00
@petermikitsh
Copy link
Contributor

When should I anticipate the arrival of the v0.8 release? Just noticed this bug.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@daffl @ekryski @petermikitsh