Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 12, 2025

The interface IAuthorizationHeaderBoundProvider was originally named for bound token scenarios but has expanded beyond that scope. Renaming to IAuthorizationHeaderProvider2 better reflects its generalized purpose.

Description

  • Interface renamed: IAuthorizationHeaderBoundProviderIAuthorizationHeaderProvider2

    • File: src/Microsoft.Identity.Abstractions/DownstreamApi/IAuthorizationHeaderBoundProvider.csIAuthorizationHeaderProvider2.cs
    • Interface definition updated in source
  • Public API surface updated across all target frameworks:

    • PublicAPI.Unshipped.txt files for net462, net8.0, net9.0, net10.0, netstandard2.0, netstandard2.1
  • Documentation updated:

    • README.md: Class diagrams and interface references
    • agents.md: Interface description
    • Structure.txt: Interface listing

Breaking Change

This is an API-breaking change for version 9.6. Safe to apply as the feature was never released.

// Before
public interface IAuthorizationHeaderBoundProvider : IAuthorizationHeaderProvider<OperationResult<...>>

// After  
public interface IAuthorizationHeaderProvider2 : IAuthorizationHeaderProvider<OperationResult<...>>

Fixes #227

Original prompt

This section details on the original issue you should resolve

<issue_title>Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider1 (API change for 9.6)</issue_title>
<issue_description>Is your feature request related to a problem? Please describe.
The interface currently named IAuthorizationHeaderBoundProvider is now recognized as being more generic than just bound tokens. The naming doesn't accurately reflect its expanded scope.

Describe the solution you'd like
Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider1 throughout the repository. This change aligns the interface name with its actual usage and increases clarity for future maintenance and contributions.

  • Update all usages, references, and documentation to the new interface name.
  • Version update: Targeting version 9.6

Describe alternatives you've considered
Keeping the original name, but this would lead to confusion as the interface responsibilities have expanded.

Additional context
This is a breaking change in API surface but is safe to apply since the impacted feature was never released.


Labels: api-change
Milestone: 9.6
</issue_description>

Comments on the Issue (you are @copilot in this section)


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

Copilot AI changed the title [WIP] Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider1 Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider1 Nov 12, 2025
Copilot AI requested a review from jmprieur November 12, 2025 18:30
Copilot finished work on behalf of jmprieur November 12, 2025 18:30
Copy link
Contributor

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

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

This is great @copilot
but we've decided to name the interface IAuthorizationHeaderProvider2 (and not IAuthorizationHeaderProvider1)

Can you please make this change?
I've updated the issue, btw: #227

Copy link
Contributor Author

Copilot AI commented Nov 12, 2025

This is great @copilot
but we've decided to name the interface IAuthorizationHeaderProvider2 (and not IAuthorizationHeaderProvider1)

...

Done! Updated the interface name to IAuthorizationHeaderProvider2 throughout the repository. All references have been updated including the interface file, PublicAPI files, README.md, agents.md, and Structure.txt. (commit b72dce9)

Copy link
Contributor

@jmprieur jmprieur left a comment

Choose a reason for hiding this comment

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

LGTM
Thanks copilot

Copilot AI changed the title Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider1 Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider2 Nov 12, 2025
Copilot AI requested a review from jmprieur November 12, 2025 18:40
Copilot finished work on behalf of jmprieur November 12, 2025 18:40
@jmprieur jmprieur marked this pull request as ready for review November 12, 2025 18:41
@jmprieur jmprieur requested a review from a team as a code owner November 12, 2025 18:41
@jmprieur jmprieur merged commit f9bac31 into main Nov 12, 2025
5 checks passed
@jmprieur jmprieur deleted the copilot/rename-iauthorizationheaderprovider1 branch November 12, 2025 18:52
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.

Rename IAuthorizationHeaderBoundProvider to IAuthorizationHeaderProvider2 (API change for 9.6)

4 participants