Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] WwwAuthenticateParameters should not expose Resource #3144

Closed
1 task
jmprieur opened this issue Feb 7, 2022 · 0 comments · Fixed by #3160
Closed
1 task

[Bug] WwwAuthenticateParameters should not expose Resource #3144

jmprieur opened this issue Feb 7, 2022 · 0 comments · Fixed by #3160
Assignees
Labels
Milestone

Comments

@jmprieur
Copy link
Contributor

jmprieur commented Feb 7, 2022

Which version of MSAL.NET are you using?
4.41

Platform
All

Other?
WWW-Authenticate parameters

Actual behavior
WwwAuthenticateParameters exposes the Resource property. It should not as the client should know which ressource it called

Expected behavior
Don't expose the Resource property any longer.

Possible solution

  /// <summary>
  /// Resource for which to request scopes.
  /// This is the App ID URI of the API that returned the WWW-Authenticate header.
  /// </summary>
  public string Resource 
  {
    [Obsolete("The client apps should know which App ID URI it requests scopes for") 
    get; 
    set; 
   }
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
3 participants