Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 10, 2025

Add CreateBoundAuthorizationHeaderAsync method signature to PublicAPI.Unshipped.txt

  • You've read the Contributor Guide and Code of Conduct.
  • You've included unit or integration tests for your change, where applicable.
  • You've included inline docs for your change, where applicable.
  • There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

Complete public API documentation for IBoundAuthorizationHeaderProvider

Description

Addresses review feedback on #232 by adding the complete method signature for CreateBoundAuthorizationHeaderAsync to the public API surface documentation.

The IBoundAuthorizationHeaderProvider interface was added with only the interface type documented in PublicAPI.Unshipped.txt. The method signature was missing, causing incomplete API surface tracking.

Changes

  • Added CreateBoundAuthorizationHeaderAsync method signature to all target framework PublicAPI.Unshipped.txt files (net8.0, net9.0, net10.0, net462, netstandard2.0, netstandard2.1)
Microsoft.Identity.Abstractions.IBoundAuthorizationHeaderProvider.CreateBoundAuthorizationHeaderAsync(Microsoft.Identity.Abstractions.DownstreamApiOptions! downstreamApiOptions, System.Security.Claims.ClaimsPrincipal? claimsPrincipal = null, System.Threading.CancellationToken cancellationToken = default(System.Threading.CancellationToken)) -> System.Threading.Tasks.Task<Microsoft.Identity.Abstractions.OperationResult<Microsoft.Identity.Abstractions.AuthorizationHeaderInformation!, Microsoft.Identity.Abstractions.AuthorizationHeaderError!>>!

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copy link
Contributor Author

Copilot AI commented Dec 10, 2025

Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • 0t3vsblobprodcus362.vsblob.vsassets.io
    • Triggering command: /usr/bin/dotnet dotnet build src/Microsoft.Identity.Abstractions/Microsoft.Identity.Abstractions.csproj (dns block)
  • ckzvsblobprodcus347.vsblob.vsassets.io
    • Triggering command: /usr/bin/dotnet dotnet build src/Microsoft.Identity.Abstractions/Microsoft.Identity.Abstractions.csproj (dns block)
  • sc4vsblobprodcus331.vsblob.vsassets.io
    • Triggering command: /usr/bin/dotnet dotnet build src/Microsoft.Identity.Abstractions/Microsoft.Identity.Abstractions.csproj (dns block)
  • vb4vsblobprodcus33.vsblob.vsassets.io
    • Triggering command: /usr/bin/dotnet dotnet build src/Microsoft.Identity.Abstractions/Microsoft.Identity.Abstractions.csproj (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Copilot AI changed the title [WIP] Update PublicAPI.Unshipped.txt to include new bound authorization header method Add CreateBoundAuthorizationHeaderAsync method signature to PublicAPI.Unshipped.txt Dec 10, 2025
Copilot AI requested a review from jmprieur December 10, 2025 00:41
@jmprieur jmprieur marked this pull request as ready for review December 10, 2025 00:45
@jmprieur jmprieur requested a review from a team as a code owner December 10, 2025 00:45
@jmprieur jmprieur merged commit 2e26cbc into iepoly/rename-bound-authorization-header-interface Dec 10, 2025
2 checks passed
@jmprieur jmprieur deleted the copilot/sub-pr-232 branch December 10, 2025 01:19
cpp11nullptr added a commit that referenced this pull request Dec 10, 2025
* Rename bound authorization header interface

* Add CreateBoundAuthorizationHeaderAsync method signature to PublicAPI.Unshipped.txt (#233)

* Initial plan

* Add CreateBoundAuthorizationHeaderAsync method signature to PublicAPI.Unshipped.txt files

Co-authored-by: jmprieur <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: jmprieur <[email protected]>

* Update MicrosoftIdentityAbstractionsVersion to 10.0.0 for breaking change (#234)

* Initial plan

* Update MicrosoftIdentityAbstractionsVersion to 10.0.0 for breaking change

Co-authored-by: jmprieur <[email protected]>

---------

Co-authored-by: copilot-swe-agent[bot] <[email protected]>
Co-authored-by: jmprieur <[email protected]>
Co-authored-by: Jean-Marc Prieur <[email protected]>

---------

Co-authored-by: Zhenya Polyvanyi <[email protected]>
Co-authored-by: Copilot <[email protected]>
Co-authored-by: jmprieur <[email protected]>
Co-authored-by: Jean-Marc Prieur <[email protected]>
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