Pass kid param through JWT::JWK.create_from

[shaun-guth-allscripts]

The underlying keypair classes already support an optional second parameter for the key id; however, the JWT::JWK interface didn't provide any mechanism for setting them via the constructor.

[sourcelevel-bot]

Hello, @shaun-guth-allscripts! This is your first Pull Request that will be reviewed by SourceLevel, an automatic Code Review service. It will leave comments on this diff with potential issues and style violations found in the code as you push new commits. You can also see all the issues found on this Pull Request on its review page. Please check our documentation for more information.

[anakinj]

Looks great. Could you be so kind and fix the RuboCop issue mentioned here: https://github.com/jwt/ruby-jwt/pull/445/checks?check_run_id=3740502864

[sourcelevel-bot]

SourceLevel has finished reviewing this Pull Request and has found:

• 2 possible new issues (including those that may have been commented here).

But beware that this branch is 1 commit behind the jwt:master branch, and a review of an up to date branch would produce more accurate results.

See more details about this review.

[shaun-guth-allscripts]

Looks great. Could you be so kind and fix the RuboCop issue mentioned here: https://github.com/jwt/ruby-jwt/pull/445/checks?check_run_id=3740502864

This should be fixed now - thanks for looking into it!