Annotate Blazor wasm auth assemblies for trimmability (#31119)

Minor cleanup when inspecting the code

src/Components/WebAssembly/Authentication.Msal/src/Microsoft.Authentication.WebAssembly.Msal.csproj

@@ -1,11 +1,12 @@
-<Project Sdk="Microsoft.NET.Sdk.Razor">
+<Project Sdk="Microsoft.NET.Sdk.Razor">
 
  <Sdk Name="Yarn.MSBuild" />
 
  <PropertyGroup>
  <TargetFramework>$(DefaultNetCoreTargetFramework)</TargetFramework>
  <Description>Authenticate your Blazor webassembly applications with Azure Active Directory and Azure Active Directory B2C</Description>
  <GenerateDocumentationFile>true</GenerateDocumentationFile>
+ <Trimmable>true</Trimmable>
  </PropertyGroup>
 
  <ItemGroup>
@@ -16,6 +17,10 @@
  <InternalsVisibleTo Include="Microsoft.Authentication.WebAssembly.Msal.Tests" />
  </ItemGroup>
 
+ <ItemGroup>
+ <Compile Include="$(SharedSourceRoot)LinkerFlags.cs" LinkBase="Shared" />
+ </ItemGroup>
+
  <PropertyGroup>
  <YarnWorkingDir>$(MSBuildThisFileDirectory)Interop\</YarnWorkingDir>
  <ResolveCurrentProjectStaticWebAssetsInputsDependsOn>

src/Components/WebAssembly/Authentication.Msal/src/MsalDefaultOptionsConfiguration.cs

@@ -44,7 +44,7 @@ public void Configure(RemoteAuthenticationOptions<MsalProviderOptions> options)
 
  public void PostConfigure(string name, RemoteAuthenticationOptions<MsalProviderOptions> options)
  {
- if (string.Equals(name, Options.DefaultName))
+ if (string.Equals(name, Options.DefaultName, StringComparison.Ordinal))
  {
  Configure(options);
  }

src/Components/WebAssembly/Authentication.Msal/src/MsalWebAssemblyServiceCollectionExtensions.cs

@@ -2,11 +2,13 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
+using System.Diagnostics.CodeAnalysis;
 using Microsoft.AspNetCore.Components.WebAssembly.Authentication;
 using Microsoft.Authentication.WebAssembly.Msal;
 using Microsoft.Authentication.WebAssembly.Msal.Models;
 using Microsoft.Extensions.DependencyInjection.Extensions;
 using Microsoft.Extensions.Options;
+using static Microsoft.AspNetCore.Internal.LinkerFlags;
 
 namespace Microsoft.Extensions.DependencyInjection
 {
@@ -48,7 +50,7 @@ public static IRemoteAuthenticationBuilder<TRemoteAuthenticationState, RemoteUse
  /// <param name="services">The <see cref="IServiceCollection"/>.</param>
  /// <param name="configure">A callback to configure the <see cref="RemoteAuthenticationOptions{MsalProviderOptions}"/>.</param>
  /// <returns>The <see cref="IServiceCollection"/>.</returns>
- public static IRemoteAuthenticationBuilder<TRemoteAuthenticationState, TAccount> AddMsalAuthentication<TRemoteAuthenticationState, TAccount>(this IServiceCollection services, Action<RemoteAuthenticationOptions<MsalProviderOptions>> configure)
+ public static IRemoteAuthenticationBuilder<TRemoteAuthenticationState, TAccount> AddMsalAuthentication<TRemoteAuthenticationState, [DynamicallyAccessedMembers(JsonSerialized)] TAccount>(this IServiceCollection services, Action<RemoteAuthenticationOptions<MsalProviderOptions>> configure)
  where TRemoteAuthenticationState : RemoteAuthenticationState, new()
  where TAccount : RemoteUserAccount
  {

src/Components/WebAssembly/WebAssembly.Authentication/src/IRemoteAuthenticationPathsProvider.cs

@@ -1,7 +1,7 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication.Internal
+namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
 {
  /// <summary>
  /// This is an internal API that supports the Microsoft.AspNetCore.Components.WebAssembly.Authentication

src/Components/WebAssembly/WebAssembly.Authentication/src/Microsoft.AspNetCore.Components.WebAssembly.Authentication.csproj

@@ -18,6 +18,10 @@
  <InternalsVisibleTo Include="Microsoft.AspNetCore.Components.WebAssembly.Authentication.Tests" />
  </ItemGroup>
 
+ <ItemGroup>
+ <Compile Include="$(SharedSourceRoot)LinkerFlags.cs" LinkBase="Shared" />
+ </ItemGroup>
+
  <PropertyGroup>
  <YarnWorkingDir>$(MSBuildThisFileDirectory)Interop\</YarnWorkingDir>
  <ResolveCurrentProjectStaticWebAssetsInputsDependsOn>

src/Components/WebAssembly/WebAssembly.Authentication/src/QueryStringHelper.cs

@@ -1,11 +1,12 @@
-// Copyright (c) .NET Foundation. All rights reserved.
+// Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
+using System.Text.Encodings.Web;
 
 namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
 {
- internal class QueryStringHelper
+ internal static class QueryStringHelper
  {
  public static string GetParameter(string queryString, string key)
  {

src/Components/WebAssembly/WebAssembly.Authentication/src/RemoteAuthenticationBuilderExtensions.cs

@@ -49,7 +49,7 @@ public static class RemoteAuthenticationBuilderExtensions
  /// <typeparam name="TAccountClaimsPrincipalFactory">The new user factory type.</typeparam>
  /// <param name="builder">The <see cref="IRemoteAuthenticationBuilder{RemoteAuthenticationState, Account}"/>.</param>
  /// <returns>The <see cref="IRemoteAuthenticationBuilder{RemoteAuthenticationState, Account}"/>.</returns>
- public static IRemoteAuthenticationBuilder<RemoteAuthenticationState, RemoteUserAccount> AddAccountClaimsPrincipalFactory<TAccountClaimsPrincipalFactory>(
+ public static IRemoteAuthenticationBuilder<RemoteAuthenticationState, RemoteUserAccount> AddAccountClaimsPrincipalFactory<[DynamicallyAccessedMembers(DynamicallyAccessedMemberTypes.PublicConstructors)] TAccountClaimsPrincipalFactory>(
  this IRemoteAuthenticationBuilder<RemoteAuthenticationState, RemoteUserAccount> builder)
  where TAccountClaimsPrincipalFactory : AccountClaimsPrincipalFactory<RemoteUserAccount> => builder.AddAccountClaimsPrincipalFactory<RemoteAuthenticationState, RemoteUserAccount, TAccountClaimsPrincipalFactory>();
  }

src/Components/WebAssembly/WebAssembly.Authentication/src/RemoteAuthenticatorViewCore.cs

@@ -5,7 +5,6 @@
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Components.Authorization;
 using Microsoft.AspNetCore.Components.Rendering;
-using Microsoft.AspNetCore.Components.WebAssembly.Authentication.Internal;
 using Microsoft.JSInterop;
 
 namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
@@ -103,9 +102,7 @@ public class RemoteAuthenticatorViewCore<TAuthenticationState> : ComponentBase w
  /// <summary>
  /// Gets or sets a default <see cref="IRemoteAuthenticationPathsProvider"/> to use as fallback if an <see cref="ApplicationPaths"/> has not been explicitly specified.
  /// </summary>
-#pragma warning disable PUB0001 // Pubternal type in public API
  [Inject] internal IRemoteAuthenticationPathsProvider RemoteApplicationPathsProvider { get; set; }
-#pragma warning restore PUB0001 // Pubternal type in public API
 
  /// <summary>
  /// Gets or sets a default <see cref="AuthenticationStateProvider"/> with the current user.

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/AccessTokenResult.cs

@@ -1,8 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System;
-
 namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
 {
  /// <summary>

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/AccountClaimsPrincipalFactory.cs

@@ -16,7 +16,6 @@ public class AccountClaimsPrincipalFactory<TAccount> where TAccount : RemoteUser
  {
  private readonly IAccessTokenProviderAccessor _accessor;
 
-#pragma warning disable PUB0001 // Pubternal type in public API
  /// <summary>
  /// Initialize a new instance of <see cref="AccountClaimsPrincipalFactory{TAccount}"/>.
  /// </summary>

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/DefaultRemoteApplicationPathsProvider.cs

@@ -1,7 +1,6 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using Microsoft.AspNetCore.Components.WebAssembly.Authentication.Internal;
 using Microsoft.Extensions.Options;
 
 namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/RemoteAuthenticationService.cs

@@ -2,12 +2,13 @@
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
 using System;
+using System.Diagnostics.CodeAnalysis;
 using System.Security.Claims;
-using System.Text.Json;
 using System.Threading.Tasks;
 using Microsoft.AspNetCore.Components.Authorization;
 using Microsoft.Extensions.Options;
 using Microsoft.JSInterop;
+using static Microsoft.AspNetCore.Internal.LinkerFlags;
 
 namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
 {
@@ -17,7 +18,7 @@ namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
  /// <typeparam name="TRemoteAuthenticationState">The state to preserve across authentication operations.</typeparam>
  /// <typeparam name="TAccount">The type of the <see cref="RemoteUserAccount" />.</typeparam>
  /// <typeparam name="TProviderOptions">The options to be passed down to the underlying JavaScript library handling the authentication operations.</typeparam>
- public class RemoteAuthenticationService<TRemoteAuthenticationState, TAccount, TProviderOptions> :
+ public class RemoteAuthenticationService<TRemoteAuthenticationState, [DynamicallyAccessedMembers(JsonSerialized)] TAccount, TProviderOptions> :
  AuthenticationStateProvider,
  IRemoteAuthenticationService<TRemoteAuthenticationState>,
  IAccessTokenProvider

src/Components/WebAssembly/WebAssembly.Authentication/src/Services/SignOutSessionStateManager.cs

@@ -13,11 +13,7 @@ namespace Microsoft.AspNetCore.Components.WebAssembly.Authentication
  public class SignOutSessionStateManager
  {
  private readonly IJSRuntime _jsRuntime;
- private static readonly JsonSerializerOptions _serializationOptions = new JsonSerializerOptions
- {
- PropertyNamingPolicy = JsonNamingPolicy.CamelCase,
- PropertyNameCaseInsensitive = true,
- };
+ private static readonly JsonSerializerOptions _serializationOptions = new JsonSerializerOptions(JsonSerializerDefaults.Web);
 
  /// <summary>
  /// Initialize a new instance of <see cref="SignOutSessionStateManager"/>.