Skip to content

Conversation

@twsouthwick
Copy link
Member

No description provided.

@twsouthwick twsouthwick requested a review from Copilot November 11, 2025 19:41
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds automatic data protection provider configuration for OWIN applications by bridging ASP.NET Core's data protection system with OWIN's security infrastructure. This enables OWIN middleware to use ASP.NET Core's data protection services without manual configuration.

  • Introduces a new SetDataProtectionProvider extension method that wraps ASP.NET Core's IDataProtectionProvider for OWIN compatibility
  • Automatically configures the OWIN data protection provider when building OWIN apps if one isn't already set
  • Updates package dependencies to include Microsoft.Owin.Security.Interop v2.3.0

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
OwinBuilder.cs Adds AddAspNetCoreDefaults method to automatically configure data protection provider from ASP.NET Core services
AppBuilderExtensions.cs New file containing extension method and shim classes for bridging ASP.NET Core and OWIN data protection APIs
Microsoft.AspNetCore.SystemWebAdapters.Owin.csproj Adds reference to Microsoft.Owin.Security.Interop package
Directory.Packages.props Defines version 2.3.0 for Microsoft.Owin.Security.Interop package
samples/Directory.Packages.props Updates Microsoft.Owin.Security.Interop from 2.1.38 to 2.3.0
Web.config Adds assembly binding redirect for System.Security.Cryptography.Xml
Program.cs Adds using directive for Microsoft.Owin.Security.DataProtection namespace

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@twsouthwick twsouthwick requested a review from mjrousos November 11, 2025 20:51
@twsouthwick twsouthwick enabled auto-merge (squash) November 11, 2025 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants