Added JWK and JWKS support

[valeriomazzeo]

This pull request adds support for JWK and JWKS.
It fixes #87

Note: only RSA keys have been implemented so far.

[dehlen]

Is there anything I can do this get this PR merged? I would love to see this integrated, however the PR is open already half a year.

[valeriomazzeo]

@dehlen you can use https://github.com/asensei/vapor-auth-jwt

[tanner0101]

Sorry for missing this earlier, we should consider adding JWK support to the next major release (currently in alpha).

The module has been renamed from JWT to JWTKit, so these files must be moved. Besides that, it seems like everything here should work against the latest version on master.

[tanner0101]

Instead of supplying an allTests array, just run swift test --generate-linuxmain before committing.

[dehlen]

Great to see things coming along. I am currently using a fork for this to work and love to be able to use the stable version here with the next release. Again I am not the author of the pull request but if there is anything I can help with...

[vzsg]

@tanner0101 as this PR was for 3.0, why not merge it there and release it as 3.1.0?

[tanner0101]

@vzsg we definitely could. It's currently targeted at master which is 4.0, but I'd be happy to merge into branch 3 as well. Of course, if we do merge into 3.0, we should also add it to 4.0 at the same time.

Looking at Swift at least, they usually merge things into master (unstable), then cherry pick to the respective stable versions they would like to support.

[vzsg]

Well, back in November 2018 when this PR was opened, master was still 3.0 :)
But I'm happy with this approach.

[valeriomazzeo]

@tanner0101 be careful when merging into master or 3. This commit vapor/jwt-kit@fb65262 has removed the additional tests execution.

[grosch]

@tanner0101 is this going to be merged into master soon?

[amkomatz]

@tanner0101 Any update on this PR? Starting development on a major application, and we could really use this :)

[valeriomazzeo]

@amkomatz the content of this pull request is included in https://github.com/asensei/vapor-auth-jwt

We've been using this since Vapor 2.

[tanner0101]

Merging this into 3-jwks branch so that we can have a local branch to test a merge into 3

[tanner0101]

Update on the situation here: vapor/jwt-kit#105 (comment)

[valeriomazzeo]

I have reverted that master merge. This is the PR as it was originally intended for Vapor 3 vapor/jwt-kit#106 .

Just as a reminder I had already implemented this for Vapor 2, but for whatever reason it wasn't ported to Vapor 3.

It would be nice if you could please incorporate this feature also in the master branch, for Vapor 4. Otherwise I will have to re-implement it once again :(

[tanner0101]

Added in JWT 3.1.0, thanks @valeriomazzeo! https://github.com/vapor/jwt-kit/releases/tag/3.1.0

[tanner0101]

JWK added to master branch here: vapor/jwt-kit#107