Skip to content
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 IAlgFactory to master (v3) branch #102

Merged
merged 9 commits into from
May 13, 2017
Merged

Conversation

abatishchev
Copy link
Member

@abatishchev abatishchev commented May 1, 2017

Copying changes from #101

abatishchev and others added 7 commits April 28, 2017 10:15
* Adding IAlgorithmFactory by default implemented by AlgorithmFactory. Using IAlgorithmFactory in JwtDecoder
* Renaming AlgorithmFactory to HMACSHAAlgorithmFactory
* Adding RSAlgorithmFactory inheriting HMACSHAAlgorithmFactory
* Adding DotSettings with abbreviations
@abatishchev abatishchev added this to the 3.0 milestone May 1, 2017
@abatishchev abatishchev self-assigned this May 1, 2017
Copy link
Contributor

@latchkostov latchkostov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me

@abatishchev abatishchev requested a review from nbarbettini May 2, 2017 03:04
@djavier
Copy link

djavier commented May 2, 2017

@abatishchev the current stable Nuget version 2.4.0 does not include the latest IAlgorithmFactory changes.

@abatishchev
Copy link
Member Author

Oh, really? How could it happen? Hmm. I'll replace it tomorrow with the right assembly. Sorry about this.

@abatishchev
Copy link
Member Author

@djavier this has been fixed, 2.4.1 is release, 2.4.0 is unlisted. Thanks for reporting!

@abatishchev
Copy link
Member Author

@blgrnboy Thanks for reviewing! I tried to check-in but GitHub settings if require a review than require by a member (@nbarbettini). Let's wait him to come back and review, I'll check-in shortly thereafter.

@djavier
Copy link

djavier commented May 2, 2017 via email

@abatishchev
Copy link
Member Author

Meanwhile I published this as 3.0.0-beta2

@abatishchev
Copy link
Member Author

@nbarbettini hey Nate, please take a look to let it in.

@nbarbettini
Copy link
Contributor

Looks good 👍 sorry for the delay @abatishchev.

@abatishchev abatishchev merged commit 8137c59 into master May 13, 2017
@abatishchev abatishchev deleted the IAlgFactory-v3 branch May 13, 2017 18:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants