Skip to content
This repository was 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
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant