Skip to content

Commit

Permalink
Remove CredentialsUri from SonarQube and SonarCloud
Browse files Browse the repository at this point in the history
  • Loading branch information
vnaskos-sonar committed Jan 27, 2025
1 parent 8b2de1e commit c995b2d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 6 deletions.
2 changes: 1 addition & 1 deletion src/Core.UnitTests/Binding/ServerConnectionTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ public void Ctor_SonarQube_SetsProperties()
sonarQube.ServerUri.Should().BeSameAs(Localhost);
sonarQube.Settings.Should().BeSameAs(serverConnectionSettings);
sonarQube.Credentials.Should().BeSameAs(credentials);
sonarQube.CredentialsUri.Should().BeSameAs(sonarQube.ServerUri);
sonarQube.CredentialsUri.ToString().Should().BeSameAs(sonarQube.Id);
}

[TestMethod]
Expand Down
6 changes: 1 addition & 5 deletions src/Core/Binding/ServerConnection.cs
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
* Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
*/

using System.IO;
using SonarQube.Client.Models;

namespace SonarLint.VisualStudio.Core.Binding;
Expand All @@ -32,7 +31,7 @@ public abstract class ServerConnection
public IConnectionCredentials Credentials { get; set; }

public abstract Uri ServerUri { get; }
public abstract Uri CredentialsUri { get; }
public Uri CredentialsUri => new(Id);

public static ServerConnection FromBoundSonarQubeProject(BoundSonarQubeProject boundProject) =>
boundProject switch
Expand All @@ -57,13 +56,11 @@ public SonarCloud(string organizationKey, ServerConnectionSettings settings = nu
: base(OrganizationKeyToId(organizationKey), settings, credentials)
{
OrganizationKey = organizationKey;
CredentialsUri = new Uri(Id);
}

public string OrganizationKey { get; }

public override Uri ServerUri => new (SonarCloudUrl);
public override Uri CredentialsUri { get; }

private static string OrganizationKeyToId(string organizationKey)
{
Expand All @@ -80,6 +77,5 @@ public sealed class SonarQube(Uri serverUri, ServerConnectionSettings settings =
: ServerConnection(serverUri?.ToString(), settings, credentials)
{
public override Uri ServerUri { get; } = serverUri;
public override Uri CredentialsUri { get; } = serverUri;
}
}

0 comments on commit c995b2d

Please sign in to comment.