microsoft-authentication-library-for-dotnet icon indicating copy to clipboard operation
microsoft-authentication-library-for-dotnet copied to clipboard

[Feature Request] Integrate DiscoverAccounts API from MSALRuntime into new broker

Open gladjohn opened this issue 2 years ago • 2 comments

Is your feature request related to a problem? Please describe. Integrate DiscoverAccounts API from MSALRuntime into new broker. Interop version starting 0.10.0 will have account discovery functionality

  • [x] Test Native Interop 0.12.0
  • [x] publish Native Interop 0.12.0 to NuGet
  • [x] Include Native Interop 0.12.0 into MSAL Broker
  • [x] Integrate the new discoverAccounts functionally into MSAL Broker
  • [x] Manual testing with Dev Apps
  • [ ] Validate other accounts scenarios
  • [ ] Add integration tests

NOTE : We may need to bump the Interop version before publishing to NuGet

Describe the solution you'd like Expose Account Discovery in the new broker implementation

Describe alternatives you've considered n/a

Additional context Add any other context or screenshots about the feature request here.

gladjohn avatar Jul 07 '22 20:07 gladjohn

Native Interop 0.10.0 Discover Accounts were missing Environment and UserName properties. On 8/11/2022 MSAL Runtime team released the Native Interop 0.11.2 which has these properties, moving this back to in-progress and will start working on this.

gladjohn avatar Aug 12 '22 16:08 gladjohn

tested and published Native Interop 0.12.0 to NuGet. Included in Broker. Found one bug where the calls gets throttled while doing getAccounts, working on identifying the issue.

gladjohn avatar Aug 17 '22 18:08 gladjohn

No longer blocked, the throttling issue has been fixed in MSAL C++

bgavrilMS avatar Nov 03 '22 13:11 bgavrilMS