-
Notifications
You must be signed in to change notification settings - Fork 689
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
Add JWT support #187
Add JWT support #187
Conversation
@miremond : we talked about it, I did it ;-) |
* @author Jerome Leleu | ||
* @since 1.8.0 | ||
*/ | ||
public class JwtGenerator<U extends UserProfile> { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What is the purpose of this class? is it only for tests? If no, maybe we should parameterized the signature or encryption methods. Otherwise we can move it to the test directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good question! This class is meant to generate a JWT from an existing user profile. The typical use case is an application where the user can login with Facebook for example and then generates a token to access web services. This was inspired by this discussion: https://groups.google.com/forum/?fromgroups#!topic/pac4j-users/kJMw8vvZBJY
Congrats @leleuj ! I give you a kudo! |
Add JWT support
generation + authentication