Merge pull request #27 from rafaelldi/fix-possible-nre

Fix possible nre
JetBrains · Jan 5, 2024 · dd8e942 · dd8e942
2 parents 17ddc45 + fdce267
commit dd8e942
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 3 deletions.
diff --git a/gradle.properties b/gradle.properties
@@ -4,7 +4,7 @@ pluginGroup = me.rafaelldi.aspire
 pluginName = aspire-plugin
 pluginRepositoryUrl = https://github.com/rafaelldi/aspire-plugin
 # SemVer format -> https://semver.org
-pluginVersion = 0.2.0
+pluginVersion = 0.2.1
 
 # Supported build number ranges and IntelliJ Platform versions -> https://plugins.jetbrains.com/docs/intellij/build-number-ranges.html
 pluginSinceBuild = 233

diff --git a/src/dotnet/aspire-session-host/Models.cs b/src/dotnet/aspire-session-host/Models.cs
@@ -11,4 +11,4 @@ internal sealed record Session(
 );
 
 [UsedImplicitly]
-internal sealed record EnvironmentVariable(string Name, string Value);
+internal sealed record EnvironmentVariable(string Name, string? Value);
diff --git a/src/dotnet/aspire-session-host/Sessions/SessionService.cs b/src/dotnet/aspire-session-host/Sessions/SessionService.cs
@@ -13,11 +13,15 @@ internal sealed class SessionService(Connection connection, ILogger<SessionServi
         var id = Guid.NewGuid();
         var stringId = id.ToString();
         var serviceName = session.Env?.FirstOrDefault(it => it.Name == TelemetryServiceName);
+        var envs = session.Env
+            ?.Where(it => it.Value is not null)
+            ?.Select(it => new EnvironmentVariableModel(it.Name, it.Value!))
+            ?.ToArray();
         var sessionModel = new SessionModel(
             stringId,
             session.ProjectPath,
             session.Debug,
-            session.Env?.Select(it => new EnvironmentVariableModel(it.Name, it.Value)).ToArray(),
+            envs,
             session.Args,
             serviceName?.Value
         );