PR feedback, polishing

SonarSource · Jan 20, 2025 · 22ccabb · 22ccabb
1 parent 1cd403d
commit 22ccabb
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 16 deletions.
diff --git a/API_CHANGES.md b/API_CHANGES.md
@@ -1,3 +1,12 @@
+# 10.14
+
+## Breaking changes
+
+* Add new method `org.sonarsource.sonarlint.core.rpc.protocol.SonarLintRpcClient#invalidToken` to notify client that WebAPI calls to SQS/SQC fails due to wrong token 
+  * Client can implement this method to offer user to change credentials for the connection to fix the problem
+  * For now notification is being sent only for 403 Forbidden HTTP response code since it's corresponds to malformed/wrong token and ignores 401 Unauthorized response code since it's a user permissions problem that has to be addressed on the server
+  * Also once notification sent, backend doesn't attempt to send any requests to server anymore until credentials changed
+
 # 10.13
 
 ## Breaking changes

diff --git a/backend/core/src/main/java/org/sonarsource/sonarlint/core/connection/ServerConnection.java b/backend/core/src/main/java/org/sonarsource/sonarlint/core/connection/ServerConnection.java
@@ -28,7 +28,6 @@
 import org.sonarsource.sonarlint.core.rpc.protocol.client.sync.InvalidTokenParams;
 import org.sonarsource.sonarlint.core.serverapi.ServerApi;
 import org.sonarsource.sonarlint.core.serverapi.exception.ForbiddenException;
-import org.sonarsource.sonarlint.core.serverapi.exception.UnauthorizedException;
 
 public class ServerConnection {
 
@@ -63,9 +62,6 @@ public <T> T withClientApiAndReturn(Function<ServerApi, T> serverApiConsumer) {
     } catch (ForbiddenException e) {
       state = ConnectionState.INVALID_CREDENTIALS;
       notifyClientAboutWrongTokenIfNeeded();
-    } catch (UnauthorizedException e) {
-      state = ConnectionState.MISSING_PERMISSION;
-      notifyClientAboutWrongTokenIfNeeded();
     }
     return null;
   }
@@ -78,9 +74,6 @@ public void withClientApi(Consumer<ServerApi> serverApiConsumer) {
     } catch (ForbiddenException e) {
       state = ConnectionState.INVALID_CREDENTIALS;
       notifyClientAboutWrongTokenIfNeeded();
-    } catch (UnauthorizedException e) {
-      state = ConnectionState.MISSING_PERMISSION;
-      notifyClientAboutWrongTokenIfNeeded();
     }
   }
 

diff --git a/backend/server-connection/src/main/proto/sonarlint.proto b/backend/server-connection/src/main/proto/sonarlint.proto
@@ -100,10 +100,6 @@ message LastEventPolling {
   int64 last_event_polling = 1;
 }
 
-message LastWebApiErrorNotification {
-  int64 last_wrong_token_notification = 1;
-}
-
 enum NewCodeDefinitionMode {
   UNKNOWN = 0;
   NUMBER_OF_DAYS = 1;

diff --git a/medium-tests/src/test/java/mediumtest/synchronization/ConnectionSyncMediumTests.java b/medium-tests/src/test/java/mediumtest/synchronization/ConnectionSyncMediumTests.java
@@ -23,8 +23,6 @@
 import java.time.temporal.ChronoUnit;
 import java.util.List;
 import java.util.concurrent.ExecutionException;
-import org.junit.jupiter.params.ParameterizedTest;
-import org.junit.jupiter.params.provider.ValueSource;
 import org.sonarsource.sonarlint.core.commons.RuleType;
 import org.sonarsource.sonarlint.core.commons.api.TextRange;
 import org.sonarsource.sonarlint.core.rpc.protocol.backend.connection.config.DidChangeCredentialsParams;
@@ -133,9 +131,9 @@ void it_should_sync_when_credentials_are_updated(SonarLintTestHarness harness) {
       "Synchronizing project branches for project 'projectKey'"));
   }
 
-  @ParameterizedTest
-  @ValueSource(ints = {401, 403})
-  void it_should_notify_client_if_invalid_token(Integer status) {
+  @SonarLintTest
+  void it_should_notify_client_if_invalid_token() {
+    var status = 403;
     var client = newFakeClient()
       .withCredentials(CONNECTION_ID, "user", "pw")
       .build();