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.

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

Config options #70

Merged
merged 9 commits into from Feb 19, 2016

Conversation

Projects
None yet
1 participant
@ekryski
Copy link
Member

ekryski commented Feb 19, 2016

This has some breaking changes for the config options but most of them have been undocumented. It goes along withe new documentation changes I have in feathersjs/docs#30.

In this PR:

  • If you use a default token config or follow convention you won't have to pass it to the verifyToken hook (Closes #69).
  • Support for a failRedirect (Closes #62)
  • Disable our successRedirect route if a custom one is passedin (Closes #61)

ekryski added some commits Feb 19, 2016

ekryski added a commit that referenced this pull request Feb 19, 2016

@ekryski ekryski merged commit 666b87e into master Feb 19, 2016

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@ekryski ekryski deleted the config-options branch Feb 19, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.