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

Add unauthorized_entity to authenticate_for's default for called by callback #230

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

sakuraineed
Copy link

@sakuraineed sakuraineed commented Oct 28, 2018

fixes #228

I'm having the same problem described in issue.
This fix would make expected behaviour as the doc suggests.

…allback (nsarno#228)

- Add failing test authenticate_for namespaced model
…allback (nsarno#228)

- Add unauthorized_entity to authenticate_for's default

- Remove unused argument from unauthorized_entity
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Namespaced model could cause a severe security issue if implemented as the doc suggests
1 participant