added 2 TODO notes to better explain changes

git-svn-id: https://dotnetopenid.googlecode.com/svn/trunk@51 01efa1a6-402a-0410-b0ae-47b76eba00f0
DotNetOpenAuth · Apr 4, 2007 · 3e826db · 3e826db
1 parent 598341c
commit 3e826db
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/source/Janrain.OpenId/Consumer/OpenIdLogin.cs b/source/Janrain.OpenId/Consumer/OpenIdLogin.cs
@@ -98,7 +98,10 @@ protected override void InitializeControls()
             uriFormatValidator = new RegularExpressionValidator();
             uriFormatValidator.ErrorMessage = uriFormatTextDefault + requiredTextSuffix;
             uriFormatValidator.ValidationExpression = @"http(s)?://([\w-]+\.)+[\w-]+(/[\w- ./?%&=]*)?";
+
+		    // TODO: (willem.muller) Disabling the validator  on 04/04/2007 because the above validation expression needs to be updated as it doesn't allow for URLs with IP's
             uriFormatValidator.Enabled = false;
+
             uriFormatValidator.Display = ValidatorDisplay.Dynamic;
             uriFormatValidator.ControlToValidate = WrappedTextBox.ID;
             uriFormatValidator.ValidationGroup = validationGroupDefault;

diff --git a/source/Janrain.OpenId/Server/CheckIdRequest.cs b/source/Janrain.OpenId/Server/CheckIdRequest.cs
@@ -416,7 +416,9 @@ public bool TrustRootValid
                 if (tr == null)
                     throw new MalformedTrustRoot(null, _trust_root);
 
+                // TODO (willem.muller) - The trust code on 04/04/07 is dodgy. So returing true so all trust roots are valid for now.
                 return true;
+
                 //return tr.ValidateUrl(_return_to);
             }
         }