Skip to content
This repository has been archived by the owner on Apr 23, 2019. It is now read-only.

Feathers authentication 1.0 compatible client #7

Merged
merged 12 commits into from
Nov 18, 2016
Merged

Feathers authentication 1.0 compatible client #7

merged 12 commits into from
Nov 18, 2016

Conversation

ekryski
Copy link
Member

@ekryski ekryski commented Oct 29, 2016

Summary

This makes the client compatible with feathers-authentication v1. Notable changes:

  • When calling .authenticate() you pass strategy instead of type. The strategy must match a registered passport strategy that was registered server side.
  • Move to supporting accessToken instead of token
  • The user is not longer automatically returned. Left up to the developer to fetch
  • unauthorized and authenticated events have been removed in the socket auth flow in favour of using the callback mechanism.
  • client will automatically try and authenticate if an accessToken exists in localstorage

Tasks Remaining

  • Get tests passing
  • Update docs
  • Migration guide
  • Socket auto-authenticate on transport upgrade
  • Socket auto-authenticate on re-connect

Other Information

Related to feathersjs-ecosystem/authentication#336

@ekryski
Copy link
Member Author

ekryski commented Nov 18, 2016

@marshallswain this is ready for release I think. I was thinking we can add the socket upgrade stuff as a 1.1 release.

@ekryski ekryski merged commit 2720e99 into master Nov 18, 2016
@ekryski ekryski deleted the 1.0 branch November 18, 2016 17:32
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.

None yet

1 participant