Fixes issue with no Authentication attribute in controllers (#42)

mjamry · web-flow · commit f30222191693 · 2021-05-11T16:08:04.000Z
diff --git a/nanoFramework.WebServer/WebServer.cs b/nanoFramework.WebServer/WebServer.cs
@@ -555,7 +555,7 @@ private void StartListener()
 
                         // Check auth first
                         mustAuthenticate = route.Authentication != null && route.Authentication.AuthenticationType != AuthenticationType.None;
-                        isAuthOk = false;
+                        isAuthOk = !mustAuthenticate;
 
                         if (mustAuthenticate)
                         {
@@ -578,7 +578,7 @@ private void StartListener()
                             }
                         }
 
-                        if (mustAuthenticate && isAuthOk)
+                        if (isAuthOk)
                         {
                             route.Callback.Invoke(null, new object[] { new WebServerEventArgs(context) });
                         }

Original file line number	Diff line number	Diff line change
`@@ -555,7 +555,7 @@ private void StartListener()`
`555`	`555`
`556`	`556`	`// Check auth first`
`557`	`557`	`mustAuthenticate = route.Authentication != null && route.Authentication.AuthenticationType != AuthenticationType.None;`
`558`		`- isAuthOk = false;`
	`558`	`+ isAuthOk = !mustAuthenticate;`
`559`	`559`
`560`	`560`	`if (mustAuthenticate)`
`561`	`561`	`{`
`@@ -578,7 +578,7 @@ private void StartListener()`
`578`	`578`	`}`
`579`	`579`	`}`
`580`	`580`
`581`		`- if (mustAuthenticate && isAuthOk)`
	`581`	`+ if (isAuthOk)`
`582`	`582`	`{`
`583`	`583`	`route.Callback.Invoke(null, new object[] { new WebServerEventArgs(context) });`
`584`	`584`	`}`