Fix some build warnings.

sevensolutions · Aug 23, 2024 · b2ad89a · b2ad89a
1 parent 943c16e
commit b2ad89a
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 5 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -13,13 +13,13 @@ jobs:
 
  steps:
  - name: Checkout
- uses: actions/checkout@v3
+ uses: actions/checkout@v4
  with:
  fetch-depth: 0
 
  # Install the .NET Core workload
  - name: Install .NET Core
- uses: actions/setup-dotnet@v3
+ uses: actions/setup-dotnet@v4
  with:
  dotnet-version: 8.0.x
 
@@ -35,7 +35,7 @@ jobs:
 
  # Upload the artifact
  - name: Upload build artifacts
- uses: actions/upload-artifact@v3
+ uses: actions/upload-artifact@v4
  with:
  name: nomad_iis
  path: ./dist
diff --git a/src/NomadIIS/Services/IisTaskHandle.cs b/src/NomadIIS/Services/IisTaskHandle.cs
@@ -89,7 +89,7 @@ public async Task<DriverStateV1> RunAsync ( ILogger<DriverService> logger, TaskC
  throw new ArgumentException( "Defining a root application with an empty alias is not allowed when using a target_website." );
  }
 
- await _owner.LockAsync( async serverManager =>
+ await _owner.LockAsync( serverManager =>
  {
  // Get a new port for the UDP logger
  if ( config.EnableUdpLogging )
@@ -147,6 +147,8 @@ await _owner.LockAsync( async serverManager =>
  }
 
  _state.ApplicationAliases = config.Applications.Select( x => x.Alias ).ToList();
+
+ return Task.CompletedTask;
  } );
  }
  catch ( Exception ex )
@@ -485,7 +487,7 @@ private static ApplicationPool CreateApplicationPool ( ServerManager serverManag
 
  if ( udpLocalPort is not null && udpRemotePort is not null )
  {
- AddEnvironmentVariable( envVarsCollection, "NOMAD_STDOUT_UDP_REMOTE_PORT", udpRemotePort.ToString() );
+ AddEnvironmentVariable( envVarsCollection, "NOMAD_STDOUT_UDP_REMOTE_PORT", udpRemotePort.Value.ToString() );
  AddEnvironmentVariable( envVarsCollection, "NOMAD_STDOUT_UDP_LOCAL_PORT", udpLocalPort.Value.ToString() );
  }