Skip to content

Conversation

@jmprieur
Copy link
Collaborator

@jmprieur jmprieur commented Jan 7, 2025

Fix Null Reference Exception in OwinTokenAcquirerFactory + other OWIN cleanup

This pull request addresses several improvements and bug fixes for the Microsoft.Identity.Web.OWIN package, specifically:

  1. Fixes a null reference exception in OwinTokenAcquirerFactory (MISE - OwinTokenAcquirerFactory Null exception during startup #3181)
  2. Removes the need to provide the MSAL redirect URI in web applications.
  3. Updates the binding redirect in the OWIN samples.

#3181

- Removes the need to provide the MSAL redirect URI in web apps
- Updates the binding redirect in the OWIN samples
@jmprieur jmprieur requested a review from a team as a code owner January 7, 2025 03:17
@github-actions
Copy link

github-actions bot commented Jan 7, 2025

Summary

Summary
Generated on: 1/7/2025 - 3:33:29 AM
Coverage date: 1/7/2025 - 3:32:09 AM - 1/7/2025 - 3:33:15 AM
Parser: MultiReport (2x Cobertura)
Assemblies: 0
Classes: 0
Files: 0
Line coverage:
Covered lines: 0
Uncovered lines: 0
Coverable lines: 0
Total lines: 0
Covered branches: 0
Total branches: 0
Method coverage: Feature is only available for sponsors

Coverage

No assemblies have been covered.

@github-actions
Copy link

github-actions bot commented Jan 7, 2025

Summary

Summary
Generated on: 1/7/2025 - 4:31:45 AM
Coverage date: 1/7/2025 - 4:30:31 AM - 1/7/2025 - 4:31:40 AM
Parser: MultiReport (2x Cobertura)
Assemblies: 0
Classes: 0
Files: 0
Line coverage:
Covered lines: 0
Uncovered lines: 0
Coverable lines: 0
Total lines: 0
Covered branches: 0
Total branches: 0
Method coverage: Feature is only available for sponsors

Coverage

No assemblies have been covered.

@jmprieur jmprieur merged commit 2dd962b into master Jan 7, 2025
5 checks passed
@jmprieur jmprieur deleted the Fix3181 branch January 7, 2025 04:34
This was referenced Aug 1, 2025
This was referenced Nov 3, 2025
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.

4 participants