Skip to content

Conversation

@ameyapat
Copy link
Contributor

…ting BARTs

Proposed changes

Add a property in request parameters so that calling application can indicate to broker that it is capable of requesting and handling BARTs (Bound App Refresh Tokens)

Type of change

  • Feature work
  • Bug fix
  • Documentation
  • Engineering change
  • Test
  • Logging/Telemetry

Risk

  • High – Errors could cause MAJOR regression of many scenarios. (Example: new large features or high level infrastructure changes)
  • Medium – Errors could cause regression of 1 or more scenarios. (Example: somewhat complex bug fixes, small new features)
  • Small – No issues are expected. (Example: Very small bug fixes, string changes, or configuration settings changes)

Additional information

Copilot AI review requested due to automatic review settings October 23, 2025 02:12
@ameyapat ameyapat requested a review from a team as a code owner October 23, 2025 02:12
Copy link
Contributor

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 support for Bound App Refresh Tokens (BARTs) by introducing a new property and constants that allow calling applications to indicate their capability to request and handle BARTs.

  • Adds isBoundAppRefreshTokenRequested property to MSIDRequestParameters for BART capability signaling
  • Introduces two new constants: MSID_BOUND_RT_EXCHANGE for OAuth2 operations and MSID_BOUND_RT_REDEEM for general BART operations

Reviewed Changes

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

Show a summary per file
File Description
IdentityCore/src/parameters/MSIDRequestParameters.h Adds the isBoundAppRefreshTokenRequested property declaration
IdentityCore/src/MSIDOAuth2Constants.m Implements the MSID_BOUND_RT_EXCHANGE constant
IdentityCore/src/MSIDOAuth2Constants.h Declares the MSID_BOUND_RT_EXCHANGE constant
IdentityCore/src/MSIDConstants.m Implements the MSID_BOUND_RT_REDEEM constant
IdentityCore/src/MSIDConstants.h Declares the MSID_BOUND_RT_REDEEM constant

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ameyapat ameyapat merged commit 57ad23f into dev Nov 24, 2025
16 of 18 checks passed
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.

3 participants