Fix authentication security hole with external providers

James Moger

2014-03-18 b4a63aad7f56486c164a15ae2477bcd251b0bb1b

 src/test/java/com/gitblit/tests/GitBlitTest.java

@@ -70,13 +70,13 @@
      UserModel user = new UserModel("james");

      user.displayName = "James Moger";



      assertTrue(user.is("James", null));

      assertTrue(user.is("James", ""));

      assertTrue(user.is("JaMeS", "anything"));

      assertFalse(user.is("James", null));

      assertFalse(user.is("James", ""));

      assertFalse(user.is("JaMeS", "anything"));



      assertTrue(user.is("james moger", null));

      assertTrue(user.is("james moger", ""));

      assertTrue(user.is("james moger", "anything"));

      assertFalse(user.is("james moger", null));

      assertFalse(user.is("james moger", ""));

      assertFalse(user.is("james moger", "anything"));



      assertFalse(user.is("joe", null));

      assertFalse(user.is("joe", ""));

@@ -156,7 +156,7 @@
      assertEquals(5, settings().getInteger("realm.userService", 5));



      assertTrue(settings().getBoolean("git.enableGitServlet", false));

      assertEquals(GitBlitSuite.USERSCONF.getAbsolutePath(), settings().getString("realm.userService", null));

      assertEquals("src/test/config/test-users.conf", settings().getString("realm.userService", null));

      assertEquals(5, settings().getInteger("realm.minPasswordLength", 0));

      List<String> mdExtensions = settings().getStrings("web.markdownExtensions");

      assertTrue(mdExtensions.size() > 0);

@@ -172,7 +172,7 @@


   @Test

   public void testAuthentication() throws Exception {

      assertTrue(session().authenticate("admin", "admin".toCharArray()) != null);

      assertTrue(authentication().authenticate("admin", "admin".toCharArray()) != null);

   }



   @Test