<p dir="ltr">Hi Miguel, </p>
<p dir="ltr">I did look at the Membership stuff, but it looked to me that the names "made sense", therefore didn't think it was decompiled.</p>
<p dir="ltr">I haven't actually looked at that much so I'll implement it this week.</p>
<p dir="ltr">@Mike, mail me when you're done if you want and I'll review it.</p>
<p dir="ltr">As for the properties, although they should do something to the generated urls, simply adding them should surely be a valid pull?  the issue at the moment is that without them, you get an exception even if it should be false.  I actually think that these are used by other classes when generating urls, not the route collection itself, but I don't know for sure.  Considering that adding them is very low risk, can we not just accept the pull and ask for further work.</p>
<p dir="ltr">Thanks for looking at these Miguel.</p>
<p dir="ltr">Martin</p>
<div class="gmail_quote">On 20 Oct 2014 19:20, "Miguel de Icaza" <<a href="mailto:miguel@xamarin.com">miguel@xamarin.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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>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><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>_______________________________________________<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>
</blockquote></div>