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

Implement MachineKey.Protect and MachineKey.Unprotect #1349

Merged
merged 1 commit into from Dec 16, 2014

Conversation

@martinjt
Copy link
Contributor

martinjt commented Oct 16, 2014

This is to implement 2 missing methods on the MachineKey class that are being used by the new version of the aspnetwebstack.

Thanks go to my 2 year old for the test string used in the unit test.

Both me, and my 2 year old licence this under MIT/X11.

@monoadmin

This comment has been minimized.

Copy link

monoadmin commented Oct 16, 2014

Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message.

@migueldeicaza

This comment has been minimized.

Copy link
Member

migueldeicaza commented Oct 20, 2014

It looks like MachineKeySectionUtils already handles different encryption methods for you, so the TODO can be removed.

@martinjt martinjt force-pushed the martinjt:MachineKeyProtect branch to 8302b57 Nov 3, 2014
tritao added a commit that referenced this pull request Dec 16, 2014
Implement MachineKey.Protect and MachineKey.Unprotect
@tritao tritao merged commit 5742335 into mono:master Dec 16, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.