From: Sébastien Lesaint <sebastien.lesaint@sonarsource.com>
Date: Fri, 16 Dec 2016 15:33:23 +0000 (+0100)
Subject: SONAR-8416 fix login failure log at OAuth2 successful authent
X-Git-Tag: 6.2.1~4
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=refs%2Fpull%2F1475%2Fhead;p=sonarqube.git

SONAR-8416 fix login failure log at OAuth2 successful authent
---

diff --git a/server/sonar-server/src/main/java/org/sonar/server/authentication/UserSessionInitializer.java b/server/sonar-server/src/main/java/org/sonar/server/authentication/UserSessionInitializer.java
index f63d4efd7d8..ea4b5da9097 100644
--- a/server/sonar-server/src/main/java/org/sonar/server/authentication/UserSessionInitializer.java
+++ b/server/sonar-server/src/main/java/org/sonar/server/authentication/UserSessionInitializer.java
@@ -61,7 +61,7 @@ public class UserSessionInitializer {
     "/batch/index", "/batch/file",
     "/maintenance/*",
     "/setup/*",
-    "/sessions/*",
+    "/sessions/*", "/oauth2/callback/*",
     "/api/system/db_migration_status", "/api/system/status", "/api/system/migrate_db",
     "/api/server/index", "/api/server/setup", "/api/server/version",
     AUTH_LOGIN_URL, AUTH_VALIDATE_URL);
diff --git a/server/sonar-server/src/test/java/org/sonar/server/authentication/UserSessionInitializerTest.java b/server/sonar-server/src/test/java/org/sonar/server/authentication/UserSessionInitializerTest.java
index b57103037e9..c51193c47b8 100644
--- a/server/sonar-server/src/test/java/org/sonar/server/authentication/UserSessionInitializerTest.java
+++ b/server/sonar-server/src/test/java/org/sonar/server/authentication/UserSessionInitializerTest.java
@@ -103,6 +103,8 @@ public class UserSessionInitializerTest {
     assertPathIsIgnored("/setup/index");
     assertPathIsIgnored("/sessions/new");
     assertPathIsIgnored("/sessions/logout");
+    assertPathIsIgnored("/oauth2/callback/github");
+    assertPathIsIgnored("/oauth2/callback/foo");
     assertPathIsIgnored("/api/system/db_migration_status");
     assertPathIsIgnored("/api/system/status");
     assertPathIsIgnored("/api/system/migrate_db");