<div dir="ltr">Hello,<div><br><div class="gmail_extra"><div class="gmail_quote"><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><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><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><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><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><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><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><div>Simple, merged.</div><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><div>Simple, merged.</div><div><br></div><div>Miguel.</div></div></div></div></div>