-
Notifications
You must be signed in to change notification settings - Fork 44
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
Adding ECDSA JWT Functions #4
Adding ECDSA JWT Functions #4
Conversation
secret_key.generate_key | ||
public_key = OpenSSL::PKey::EC.new secret_key | ||
public_key.private_key = nil | ||
|
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.
Trailing whitespace detected.
d10edd0
to
3453b31
Compare
end | ||
end | ||
end |
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.
Final newline missing.
3453b31
to
54d803b
Compare
Doorkeeper::JWT.configuration.secret_key | ||
end | ||
def secret_key | ||
return secret_key_file if !secret_key_file.nil? |
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.
Favor unless
over if
for negative conditions.
Why: * This allows anyone to use any of the three ecdsa encryption functions
54d803b
to
4d2ae34
Compare
@chriswarren, not sure if this is something that you'd want in |
@chriswarren any chance you can take a look at this? |
Sorry for the delay on getting to this - it looks great. Thanks for adding it. I'll merge it in and get it pushed up to RubyGems tonight. |
Adding ECDSA JWT Functions
Why: