Added changes to support Azure functions #856
Conversation
src/Microsoft.Identity.Web/AzureFunctionsAuthenticationHttpContextExtension.cs
Show resolved
Hide resolved
src/Microsoft.Identity.Web/AzureFunctionsAuthenticationHttpContextExtension.cs
Outdated
Show resolved
Hide resolved
src/Microsoft.Identity.Web/AzureFunctionsAuthenticationHttpContextExtension.cs
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Thanks @maliksahil
LGTM
- I suggested a change in the XML comment for the method.
- @jennyf19, I think we should also add the test app?
| public static class AzureFunctionsAuthenticationHttpContextExtension | ||
| { | ||
| /// <summary> | ||
| /// Enables Bearer authentication for an API for use in Azure Functions. |
There was a problem hiding this comment.
What about?
Enables an Azure Function to act as/expose a protected web API, enabling bearer token authentication? Calling this method from your Azure function validates the token and exposes the identity of the user or app on behalf of which your function is called, in the HttpContext.User member, where your function can exploit it.
There was a problem hiding this comment.
@jmprieur what do you mean by add the test app?
There was a problem hiding this comment.
Hmm PR is approved, should I still go ahead and edit this description? New PR or same?
There was a problem hiding this comment.
@maliksahil go ahead and edit. as it's a fork, @jmprieur or i will have to merge it. thanks.
There was a problem hiding this comment.
@jennyf19, I meant the sample using the new API: https://github.com/maliksahil/ms-identity-azurefunctions-microsoft-identity-web
|
SonarCloud Quality Gate failed. |
Changes are in AzureFunctionsAuthenticationHttpContextExtension.cs.