Revert "TSyncSession::checkCredentials(): <requiredauth>none => accept invalid credentials"

This reverts commit b76470a04038b601a2521a7e496ccde08efc5fa9.

Lukas prefers to keep the old functionality and suggested to provide a
<logininitscript> which returns TRUE in servers which need no
credential checking. Will do that in SyncEvolution.

1 files changed, 0 insertions, 8 deletions

diff --git a/src/sysync/syncsession.cpp b/src/sysync/syncsession.cpp
index a69404b..3277d8f 100644
--- a/src/sysync/syncsession.cpp
+++ b/src/sysync/syncsession.cpp
@@ -3988,14 +3988,6 @@ bool TSyncSession::checkCredentials(const char *aUserName, const SmlCredPtr_t aC
         authok=false; // anyway, reject
         PDEBUGPRINTFX(DBG_ERROR,("Authorization failed (wrong type of creds), sending 401 + chal"));
       }
-      else if (!authok && isAuthTypeAllowed(auth_none)) {
-        // Accept all kinds of credentials, even invalid ones,
-        // when no authentication is required. The client
-        // would have been allowed to connect if it hadn't
-        // provided credentials. By accepting these invalid
-        // credentials we make the client setup easier.
-        authok=true;
-      }
       else if (!authok) {
         // auth type allowed, but auth itself not ok
         aStatusCommand.setStatusCode(401); // unauthorized, bad credentials