Change the RS256Algorithm to take in the RSA Service Provider/RSA

[glennawatson]

This fixes #164

This changes the RS256Algorithm class not to directly take in a Certificate but instead get the RSA/RSACryptoServiceProvider combination.

this is helpful in scenarios such as AWS Cognitio where they provide the modulo and exponent directly.

[abatishchev]

Thanks so much for contributing this change!
The only thing, beside the comments I left above, please bump the version. At least the minor. Or the major, if it's a breaking change, and it might be.

[abatishchev]

Argh! I'm not a pro user of GitHub reviews so messed it up. Can't see half of my own comments.

[glennawatson]

Argh! I'm not a pro user of GitHub reviews so messed it up. Can't see half of my own comments.

You can click on the "Show Resolved" button in the right hand side and i'll come up.

[abatishchev]

If only I could figure out why gated build isn't getting kicked off...

[abatishchev]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[abatishchev]

/AzurePipelines run

[azure-pipelines]

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

[glennawatson]

I am on a fork most likely. Azure DevOps tends to be security concious of external forks to avoid man in the middle attacks.

[abatishchev]

Yeap, I allow the build to run on forks. Re-invoking it via the command apparently worked. And it has succeeded.

So what would you say? Good to go?

[glennawatson]

Go for it.

[abatishchev]

Done via https://abatishchev.visualstudio.com/Jwt.Net/_releaseProgress?_a=release-pipeline-progress&releaseId=11. See https://www.nuget.org/packages/JWT/5.1.0 (should appear soon)