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

start auk auth upgrades #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

start auk auth upgrades #19

wants to merge 1 commit into from

Conversation

swarthout
Copy link
Owner

No description provided.

@swarthout swarthout mentioned this pull request May 4, 2017
@idibidiart
Copy link

@swarthout

Don't you also have to get an anonymous token on client side and pass it along with non-authenticated requests? in addition to getting auth token for authenticated requests. I thought I saw something like that in the docs.

@lobosan
Copy link

lobosan commented May 8, 2017

Hi Scott, I just cloned the auk-update branch and I found an error on line 9 of the file:
src/services/viewer/hooks/index.js
where auth.hooks.authenticate('jwt'), should be
auth.authenticate('jwt')
since the require is
const auth = require('feathers-authentication').hooks;

After I changed that, the code is running. However, I'm seeing this message on the console
Mongoose option.lean is false. Use hook.results.toObject() to convert the results to JS objects

@lobosan
Copy link

lobosan commented May 9, 2017

Another problem that I found is when I'm testing the mutations located at examples/mutations.graphql where signUp works fine but logIn gives me this error:

Route: /auth/local - Page not found

@swarthout
Copy link
Owner Author

Thanks for the help! Did feathers change the routes for auth with the new release?

@lobosan
Copy link

lobosan commented May 9, 2017

I'm looking at the documentation and it seems that the path is now "authentication" but strategy should be defined as local, also the config file should be double checked as well as the imports.

Have a look at these resources, or maybe use the cli generator with the command feathers generate authentication
https://docs.feathersjs.com/guides/chat/authentication.html
https://github.com/feathersjs/feathers-authentication-local
https://github.com/feathersjs/feathers-authentication/blob/master/docs/migrating.md
https://github.com/feathersjs/feathers-authentication

@poupryc
Copy link

poupryc commented Jan 9, 2018

Warning: latest updates of feathers packages are now distributed under @feathers scope on NPM !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants