<div dir="ltr"><div>Thanks Chris,<br><br></div>The main thing I think will be off benefit with people reviewing is simply getting that second pair of eyes to sanity check it.  Looking for basic coding standards stuff, anything obvious.  Other things like advice on separating things out etc.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On 17 October 2014 20:46, Chris Rogus <span dir="ltr"><<a href="mailto:cricel472@gmail.com" target="_blank">cricel472@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><p dir="ltr">Disclaimer: I know nothing about these classes.</p>
<p dir="ltr">That said, I firmly support the plan to bring PRs into the mailing list, and would hate to see such a great plan get no love, so I want to say that from looking at these (very small and easily digested) changes, they at least look clean and good.</p>
<p dir="ltr">From a fellow user of Mono who is interested in its success and expansion (with no say in any outcomes here), thanks for doing this.</p>
<div class="gmail_quote"><div><div class="h5">On Oct 16, 2014 6:23 PM, "Martin Thwaites" <<a href="mailto:monoforum@my2cents.co.uk" target="_blank">monoforum@my2cents.co.uk</a>> wrote:<br type="attribution"></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5"><div dir="ltr"><div><div><div>Hi all,<br><br></div>I've just put up my attempt at getting the MachineKey.Protect and Unprotect methods available in mono.  I'd be grateful if someone give it a quick review to make sure it makes sense.<br><br><a href="https://github.com/mono/mono/pull/1349" target="_blank">https://github.com/mono/mono/pull/1349</a><br><br>Couple of notes.  There is no compatibility with encrypted items that were done using Windows.  This is a barebones implementation in that it will work and be at least as secure as MachineKey.Encrypt/Decrypt (as it uses the same methods under the hood).<br><br></div>This is my first PR for mono, so please be kind.  I've tried my best to follow the coding guidelines, but I'm happy to make changes.<br><br>Thanks<br></div>Martin<br></div>
<br></div></div>_______________________________________________<br>
Mono-devel-list mailing list<br>
<a href="mailto:Mono-devel-list@lists.ximian.com" target="_blank">Mono-devel-list@lists.ximian.com</a><br>
<a href="http://lists.ximian.com/mailman/listinfo/mono-devel-list" target="_blank">http://lists.ximian.com/mailman/listinfo/mono-devel-list</a><br>
<br></blockquote></div>
</blockquote></div><br></div>