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

0.7 Release #139

Merged
merged 29 commits into from
Mar 30, 2016
Merged

0.7 Release #139

merged 29 commits into from
Mar 30, 2016

Conversation

ekryski
Copy link
Member

@ekryski ekryski commented Mar 30, 2016

This PR has a bunch more bug fixes and some pretty critical security fixes


const defaults = {
cookie: 'fathers-jwt',
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's like your dad's JWT 😉

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep. JWT on punch cards.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LOL. Man I always make that typo. Cracks me up every time.

@ekryski
Copy link
Member Author

ekryski commented Mar 30, 2016

Ok this is good to go! Just need to merge some docs PRs and update the generator to use the restrictToAuthenticated hook.

@ekryski
Copy link
Member Author

ekryski commented Mar 30, 2016

This isn't actually broken it just depends on feathersjs-ecosystem/feathers-hooks#64.

@ekryski ekryski merged commit 62c3005 into master Mar 30, 2016
@ekryski ekryski deleted the 0.7 branch March 30, 2016 20:36
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.

3 participants