Skip to content

Commit

Permalink
upgrade further projects from. NET 7 to .NET 8, removing backup cs pr…
Browse files Browse the repository at this point in the history
…ojects
  • Loading branch information
leo-schick committed Jan 13, 2024
1 parent e091e4d commit 46f31bf
Show file tree
Hide file tree
Showing 13 changed files with 12 additions and 99 deletions.
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net7.0</TargetFramework>
<TargetFramework>net8.0</TargetFramework>
<Nullable>enable</Nullable>

<IsPackable>false</IsPackable>
Expand Down
2 changes: 1 addition & 1 deletion src/Saas.Admin/Saas.Admin.Service/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -140,7 +140,7 @@ void InitializeDevEnvironment()

// For local development, use the Secret Manager feature of .NET to store a connection string
// and likewise for storing a secret for the permission-api app.
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-7.0&tabs=windows
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-8.0&tabs=windows

var appConfigurationconnectionString = builder.Configuration.GetConnectionString("AppConfig")
?? throw new NullReferenceException("App config missing.");
Expand Down
2 changes: 1 addition & 1 deletion src/Saas.Application/Saas.Application.Web/Program.cs
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ void InitializeDevEnvironment()

// For local development, use the Secret Manager feature of .NET to store a connection string
// and likewise for storing a secret for the permission-api app.
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-7.0&tabs=windows
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-8.0&tabs=windows

var appConfigurationconnectionString = builder.Configuration.GetConnectionString("AppConfig")
?? throw new NullReferenceException("App config missing.");
Expand Down

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net7.0</TargetFramework>
<TargetFramework>net8.0</TargetFramework>
<GenerateDocumentationFile>True</GenerateDocumentationFile>
<Nullable>enable</Nullable>
</PropertyGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -68,8 +68,8 @@ resource signupAdministrationWeb 'Microsoft.Web/sites@2022-03-01' = {
http20Enabled: true
keyVaultReferenceIdentity: userAssignedIdentity.id // Must specify this when using User Assigned Managed Identity. Read here: https://learn.microsoft.com/en-us/azure/app-service/app-service-key-vault-references?tabs=azure-cli#access-vaults-with-a-user-assigned-identity
detailedErrorLoggingEnabled: true
netFrameworkVersion: 'v7.0'
// linuxFxVersion: 'DOTNETCORE|7.0'
netFrameworkVersion: 'v8.0'
// linuxFxVersion: 'DOTNETCORE|8.0'
}
}
identity: {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ public class ClientAssertionSigningProvider(
{
private readonly ILogger _logger = logger;

// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-7.0
// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-8.0
private static readonly Action<ILogger, Exception> _logError = LoggerMessage.Define(
LogLevel.Error,
new EventId(1, nameof(ClientAssertionSigningProvider)),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ public class PublicX509CertificateDetailProvider(
{
private readonly ILogger _logger = logger;

// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-7.0
// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-8.0
private static readonly Action<ILogger, Exception> _logError = LoggerMessage.Define(
LogLevel.Error,
new EventId(1, nameof(PublicX509CertificateDetailProvider)),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ public class SaasApiAuthenticationProvider<TProvider, TOptions> : DelegatingHand
{
private readonly ILogger _logger;

// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-7.0
// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-8.0
private static readonly Action<ILogger, Exception> _logError = LoggerMessage.Define(
LogLevel.Error,
new EventId(1, nameof(SaasApiAuthenticationProvider<TProvider, TOptions>)),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ public class SaasGraphClientCredentialsProvider<TOptions>(
private readonly ILogger _logger = logger;
private readonly SaasApiAuthenticationProvider<ISaasMicrosoftGraphApi, TOptions> _authProvider = authProvider;

// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-7.0
// https://learn.microsoft.com/en-us/aspnet/core/fundamentals/logging/loggermessage?view=aspnetcore-8.0
private static readonly Action<ILogger, Exception> _logError = LoggerMessage.Define(
LogLevel.Error,
new EventId(1, nameof(SaasGraphClientCredentialsProvider<TOptions>)),
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ void InitializeDevEnvironment()

// For local development, use the Secret Manager feature of .NET to store a connection string
// and likewise for storing a secret for the permission-api app.
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-7.0&tabs=windows
// https://learn.microsoft.com/en-us/aspnet/core/security/app-secrets?view=aspnetcore-8.0&tabs=windows

var appConfigurationconnectionString = builder.Configuration.GetConnectionString("AppConfig")
?? throw new NullReferenceException("App config missing.");
Expand Down

This file was deleted.

2 changes: 1 addition & 1 deletion src/TestUtilities/TestUtilities.csproj
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
<Project Sdk="Microsoft.NET.Sdk">

<PropertyGroup>
<TargetFramework>net7.0</TargetFramework>
<TargetFramework>net8.0</TargetFramework>
<ImplicitUsings>enable</ImplicitUsings>
<Nullable>enable</Nullable>
</PropertyGroup>
Expand Down

0 comments on commit 46f31bf

Please sign in to comment.