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

Added new event when token is authenticated #69

Merged
merged 1 commit into from
Jun 10, 2015

Conversation

gamringer
Copy link
Contributor

With this, it is now possible to transpose elements from the JWT payload to the security context's authenticated token.

@gamringer
Copy link
Contributor Author

Tests are failing with a deprecation notice from 2.6 : /

@slashfan
Copy link
Contributor

slashfan commented Jun 5, 2015

Hi, don't worry we are on it ! Thanks for the PR by the way, it's a nice addition. We'll merge it as soon as we fix the builds on travis.

@slashfan
Copy link
Contributor

slashfan commented Jun 6, 2015

Would you mind squashing your commits into a single one ? See this for step by step instructions.

JWTProvider::$dispatcher

Added eventDispatcher to instanciation of JWTProvider
@gamringer
Copy link
Contributor Author

I've squashed it all into 1 commit. Let me know if you need anything else.

@slashfan
Copy link
Contributor

Nice thanks. We'll just have to check that injecting the event dispatcher in the JWTProvider doesn't cause any BC break before tagging it but it's ok to merge !

slashfan added a commit that referenced this pull request Jun 10, 2015
Added new event when token is authenticated
@slashfan slashfan merged commit d23b3f5 into lexik:master Jun 10, 2015
@9orky
Copy link
Contributor

9orky commented Jul 3, 2015

@slashfan Hello, I just wanted to ask when you will release a new version of bundle (1.0.11 probably) including changes introduced by this PR? Possibility to get JWT itself is very helpful ;-)

@slashfan
Copy link
Contributor

slashfan commented Jul 6, 2015

Hi, I'll try to tag it today. I'll let you know when it's done.

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.

3 participants