<div dir="ltr">Hey Mike,<div><br></div><div>I would just submit a new pull request purely with code that you wrote, excluding any third party code.</div><div><br></div><div>As for the WebRoute, like I mentioned, while it adds the properties, it looks broken, the properties *should* do something, currently they dont.</div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 20, 2014 at 12:32 PM, Mike Morano <span dir="ltr"><<a href="mailto:mmorano@mikeandwan.us" target="_blank">mmorano@mikeandwan.us</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Miguel,<div><br></div><div>I can not apologize enough for the issue you point out in regards to the poisoned patch, I had no idea it was sourced from MS.  I have been a big mono fan since 2003, and certainly was not trying to jeopardize mono.  As I identified in my commit message, I found that from another repo on github when I was trying to see if anyone had already implemented this (similar to how I found PR874 which was a pre-requisite), though will never do that again.</div><div><br></div><div>Are the files outside of the files listed in the other repo acceptable (everything except <span style="color:rgb(51,51,51);font-family:Helvetica,arial,freesans,clean,sans-serif,'Segoe UI Emoji','Segoe UI Symbol';font-size:13px;line-height:33px;background-color:rgb(247,247,247)">MembershipPasswordAttribute.cs)?</span></div><div><br></div><div>Sorry,</div><div>Mike </div><div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote"><div><div class="h5">On Mon, Oct 20, 2014 at 10:06 AM, Miguel de Icaza <span dir="ltr"><<a href="mailto:miguel@xamarin.com" target="_blank">miguel@xamarin.com</a>></span> wrote:<br></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">Hello,<div><br><div class="gmail_extra"><div class="gmail_quote"><span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>PR874 - from Chris Carroll with a few properties implemented around routes<br></div></div></blockquote><div><br></div></span><div>While the properties were added, they are not actually used for anything, this looks bogus.</div><div><br></div><div>The tests basically show that setting a boolean property back and forth is set, but likely what needs to be tested is the other methods it affects.</div><div><br></div><div>The point of these properties is to alter the behavior of the route collection.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div></div><div>PR1163 - from AerisG222 which includes a few changes around Unvalidated parameters and some other bits</div></div></blockquote><div><br></div></span><div>This patch is poisoned.</div><div><br></div><div>There were a couple of elements that looked very suspicious, like this bit (which also, does not follow the Mono coding guidelines):</div><div><br></div><div><pre style="color:rgb(0,0,0)">          int? _minRequiredPasswordLength;
                int? _minRequiredNonAlphanumericCharacters;
                string _passwordStrengthRegularExpression;

                readonly string _minRequiredPasswordLengthError = "{0} must have at least {1} characters";
                readonly string _minNonAlphanumericCharactersError = "{0} must have at least {1} special characters";
                readonly string _passwordStrengthError = "{0} is weak";
</pre></div><div><br></div><div>What are the chances of getting every internal method name to match the one in .NET I asked myself?   Close to zero.   So this was clearly decompiled and submitted.</div><div><br></div><div>Rejected.  Someone *else* that has not worked on that code will have to rewrite this, it is unacceptable to have people contribute decompiled code.</div><span><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>PR1349 - From me regarding MachineKey.Protect methods</div></div></blockquote><div><br></div></span><div>Added a comment, looks like it could go in, but we need tests for the Unprotect path.</div><div><br></div><div>And I would like those to be tested against Windows as well.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>PR1353 - From me regarding ReadEntityBodyMode (doesn't actually work, just the interface)</div></div></blockquote><div><br></div></span><div>Simple, merged.</div><span><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr"><div>PR1354 - From me regarding Request.Abort</div></div></blockquote><div><br></div></span><div>Simple, merged.</div><span><font color="#888888"><div><br></div><div>Miguel.</div></font></span></div></div></div></div>
<br></div></div><span class="">_______________________________________________<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></span></blockquote></div><br></div>
</blockquote></div><br></div>