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

Added references to Azure functions samples #41730

Merged
merged 2 commits into from
Nov 8, 2019

Conversation

maliksahil
Copy link
Contributor

Added references to Azure functions token validation examples in dotnet, python and nodejs.

Added references to Azure functions token validation examples in dotnet, python and nodejs.
@PRMerger9
Copy link
Contributor

@maliksahil : Thanks for your contribution! The author(s) have been notified to review your proposed change.

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. I left a suggestion

@@ -137,6 +137,10 @@ The JwtBearer middleware, like the OpenID Connect middleware in web apps, is dir

There can also be special validations. For example, it's possible to validate that signing keys (when embedded in a token) are trusted and that the token isn't being replayed. Finally, some protocols require specific validations.

## Token validation in Azure Functions
Copy link
Contributor

Choose a reason for hiding this comment

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

@maliksahil, I'd put this paragraph after Validators (otherwise we have an H2 here, and an unrelated H3 after?

@ktoliver ktoliver added the aq-pr-triaged tracking label for the PR review team label Oct 30, 2019
@PRMerger18
Copy link
Contributor

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@PRMerger18 PRMerger18 requested a review from jmprieur November 8, 2019 17:50
@ktoliver ktoliver merged commit 4bcd777 into MicrosoftDocs:master Nov 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants