Migrate solution to .NET 10#102
Open
Purva-Patki wants to merge 13 commits into
Open
Conversation
- Retarget all projects to net10.0 (was netstandard2.1 / net6.0 / netcoreapp3.1) - Replace legacy ASP.NET Core 2.2 package refs with FrameworkReference Microsoft.AspNetCore.App - Bump JwtBearer and System.Drawing.Common to satisfy MISE 1.35.0 transitive requirements - Remove dead 'using static HostingApplication' imports removed after ASP.NET Core 2.2 - Update CI and Azure DevOps pipelines to install .NET 10 SDK Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
|
@Purva-Patki please read the following Contributor License Agreement(CLA). If you agree with the CLA, please reply with the following information.
Contributor License AgreementContribution License AgreementThis Contribution License Agreement (“Agreement”) is agreed to by the party signing below (“You”),
|
Removes NU1510-flagged PackageReferences now supplied by the runtime/ASP.NET Core shared framework: Microsoft.AspNetCore.Hosting.Abstractions, Microsoft.AspNetCore.Http.Abstractions, Microsoft.Extensions.Logging.Debug, Microsoft.Extensions.Configuration.Binder, System.Text.Encodings.Web. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
…mps) Bump non-breaking package versions across all projects and align transitive floors surfaced by NU1605: - Azure SDKs: Identity 1.21.0, AppConfiguration 1.10.0, Storage.Blobs 12.29.1, Cosmos 3.62.0, KeyVault.Secrets 4.11.0, Security config secrets 1.5.1 - Newtonsoft.Json 13.0.4, System.Linq.Dynamic.Core 1.7.3, UnifiedRedisPlatform 1.0.4 - Microsoft.Extensions.* + JwtBearer + System.Drawing.Common -> 10.0.10 - IdentityModel.Validators 8.21.0, Identity.Client 4.84.2 (Azure.Core 1.59 floor) - NuGet.Packaging/Protocol 6.12.1 (CodeGeneration.Design 10.0.2 floor) - Test stack: Microsoft.NET.Test.Sdk 18.8.1, Moq 4.20.72, coverlet 10.0.1, MSTest.TestAdapter/Framework 4.3.2 Migrate 3 [ExpectedException] usages to Assert.ThrowsExactly/ThrowsExactlyAsync (removed in MSTest 4). Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
MISE 2.x is a breaking redesign. Changes: - Bump Microsoft.Identity.ServiceEssentials.AspNetCore + .Caching to 2.5.3 - Drop Microsoft.Identity.ServiceEssentials.Extensions.AspNetCoreMiddleware (no 2.x; middleware folded into the AspNetCore metapackage) - ServicesExtensions.AddAuthentication: AddMiseWithDefaultAuthentication -> AddMiseWithDefaultModules; options now set via MiseOptions.AzureAd (Authority/Audiences/ClientId/Instance/TenantId) - Auth scheme constant S2SAuthenticationDefaults.AuthenticationScheme -> MiseAuthenticationDefaults.AuthenticationScheme (same value 'S2SAuthentication'; MISE 2.x no longer pulls the S2S package transitively) - UseMise namespace: ...Extensions.AspNetCoreMiddleware -> Microsoft.Identity.ServiceEssentials Builds clean; all 219 unit tests pass. NOTE: token validation/auth wiring is not exercised by unit tests - integration/auth testing recommended before merge. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
…R 2003349) Aligns the MISE 2.x auth setup with the reviewed CE-FS-FExP-PageTour MISE 2.X compliance implementation: - Configure MiseInboundPolicyOptions with a Bearer protocol accepting both app and user tokens (AppToken/UserToken/AllowMissingIdTypeClaim). MISE 2.x needs an explicit inbound policy - AzureAd audiences alone do not validate inbound tokens. - Set AzureAd.Audience (single) and expand Audiences to include api:// URI forms and the client id (v1.0/v2.0 token audience formats), plus AdditionalAudiences in both raw and api:// forms. - Drop explicit Authority; rely on Instance + TenantId (MISE-derived authority), matching the reference. - Remove redundant app.UseMise(); the AddMiseWithDefaultModules handler validates inbound tokens via UseAuthentication + AuthenticateAsync (reference does the same). OpenTelemetry.Exporter.Geneva 1.11.2 (the transitive dep the reference pinned) is already resolved automatically here (no central package pinning in this repo). Builds clean; all 219 unit tests pass. Auth behavior still needs integration validation before merge. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Dropped the Swashbuckle.AspNetCore package reference, the AddSwagger service registration and its UseSwagger/UseSwaggerUI middleware, and the associated Microsoft.OpenApi usings. Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
…tion 7.2.0 -> 11.0.2 Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Microsoft.Azure.AppConfiguration.AspNetCore and Microsoft.Extensions.Configuration.AzureAppConfiguration both to 8.5.0. Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Bumped Microsoft.FeatureManagement 2.6.1 -> 4.6.0 (Domain + API) and Microsoft.FeatureManagement.AspNetCore 2.4.0 -> 4.6.0 (API). Filter API (IFeatureFilter/FeatureFilterEvaluationContext/IFilterParametersBinder) remained source-compatible. Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Bumped across all 7 project/test references to the latest public stable on nuget.org. The 16.0.14415.15073 reported by 'dotnet list --outdated' is an internal-feed-only build not available on nuget.org, so it was intentionally not used to keep this public repo restorable from nuget.org. Build clean, all 219 tests pass. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
Migrated GraphGroupVerificationService to the Graph SDK v6 (Kiota) API: - IGraphServiceClient -> GraphServiceClient - Replaced removed DelegateAuthenticationProvider with a custom IAuthenticationProvider (MsalConfidentialClientAuthenticationProvider) that sets the bearer token from the MSAL confidential client - Replaced removed .Request()/NextPageRequest fluent+paging with GetAsync() + PageIterator<DirectoryObject, DirectoryObjectCollectionResponse> - Models now from Microsoft.Graph.Models (User, DirectoryObject) Build clean, all 219 tests pass. Graph request/paging/auth paths are not unit-tested (require live Graph); integration verification recommended. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 6c975713-01f9-4b4a-a6d9-3fc334559698
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Retarget all projects to net10.0 (was netstandard2.1 / net6.0 / netcoreapp3.1)
Replace legacy ASP.NET Core 2.2 package refs with FrameworkReference Microsoft.AspNetCore.App
Bump JwtBearer and System.Drawing.Common to satisfy MISE 1.35.0 transitive requirements
Remove dead 'using static HostingApplication' imports removed after ASP.NET Core 2.2
Update CI and Azure DevOps pipelines to install .NET 10 SDK
Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com