Skip to content

Commit 97a84e4

Browse files
committed
Fix broken build (this branch uses an old Sonar version)
1 parent 6ada975 commit 97a84e4

5 files changed

Lines changed: 12 additions & 8 deletions

File tree

src/Configuration/src/Abstractions/CompositeConfigurationProvider.cs

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,9 @@ public void Set(string key, string? value)
9797

9898
LogSet(GetType().Name, key, value);
9999

100-
if (ConfigurationRoot != null)
101-
{
102-
ConfigurationRoot[key] = value;
103-
}
100+
#pragma warning disable S1121 // Assignments should not be made from within sub-expressions
101+
ConfigurationRoot?[key] = value;
102+
#pragma warning restore S1121 // Assignments should not be made from within sub-expressions
104103
}
105104

106105
public void Dispose()

src/Connectors/src/Connectors/DynamicTypeAccess/ConnectorFactoryShim.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ private static TypeAccessor MakeGenericTypeAccessor(Type connectionType)
6060

6161
public ConnectorShim<TOptions> Get(string serviceBindingName)
6262
{
63-
object instance = InstanceAccessor.InvokeMethodOverload(nameof(ConnectorFactory<TOptions, object>.Get), true, [typeof(string)], serviceBindingName)!;
63+
object instance = InstanceAccessor.InvokeMethodOverload(nameof(ConnectorFactory<,>.Get), true, [typeof(string)], serviceBindingName)!;
6464

6565
return new ConnectorShim<TOptions>(_connectionType, instance);
6666
}

src/Connectors/src/Connectors/DynamicTypeAccess/ConnectorShim.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ private static InstanceAccessor CreateAccessor(Type connectionType, object insta
2525

2626
public object GetConnection()
2727
{
28-
return InstanceAccessor.InvokeMethod(nameof(Connector<TOptions, object>.GetConnection), true)!;
28+
return InstanceAccessor.InvokeMethod(nameof(Connector<,>.GetConnection), true)!;
2929
}
3030

3131
public void Dispose()

src/Discovery/src/Eureka/AppInfo/ApplicationInfo.cs

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ public sealed class ApplicationInfo
1717
private readonly ConcurrentDictionary<string, InstanceInfo> _instanceMap = new();
1818

1919
public string Name { get; }
20-
public IReadOnlyList<InstanceInfo> Instances => new List<InstanceInfo>(_instanceMap.Values);
20+
public IReadOnlyList<InstanceInfo> Instances => GetInstancesSnapshot();
2121

2222
internal ApplicationInfo(string name)
2323
: this(name, Array.Empty<InstanceInfo>())
@@ -38,6 +38,11 @@ internal ApplicationInfo(string name, ICollection<InstanceInfo> instances)
3838
}
3939
}
4040

41+
private List<InstanceInfo> GetInstancesSnapshot()
42+
{
43+
return [.. _instanceMap.Values];
44+
}
45+
4146
internal InstanceInfo? GetInstance(string instanceId)
4247
{
4348
ArgumentException.ThrowIfNullOrWhiteSpace(instanceId);

src/Management/src/Endpoint/Actuators/RouteMappings/AspNetEndpointProvider.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -167,7 +167,7 @@ private static IEnumerable<AspNetEndpoint> FromActuatorEndpoint(RouteEndpoint en
167167
if (endpointOptions != null)
168168
{
169169
string displayName = endpoint.DisplayName ?? string.Empty;
170-
MethodInfo handlerMethod = typeof(EndpointMiddleware<,>).GetMethod(nameof(EndpointMiddleware<object, object>.InvokeAsync))!;
170+
MethodInfo handlerMethod = typeof(EndpointMiddleware<,>).GetMethod(nameof(EndpointMiddleware<,>.InvokeAsync))!;
171171
var metadataProvider = endpoint.Metadata.GetMetadata<ActuatorMetadataProvider>()!;
172172

173173
foreach (string httpMethod in endpointOptions.GetSafeAllowedVerbs())

0 commit comments

Comments
 (0)