Implement Airtable OAuth provider #895
Conversation
|
Also, I've tested the Airtable provider against the production service and can confirm that it works fine. |
src/AspNet.Security.OAuth.Airtable/AirtableAuthenticationHandler.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.Airtable/AirtableAuthenticationHandler.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.Airtable/AirtableAuthenticationHandler.cs
Outdated
Show resolved
Hide resolved
src/AspNet.Security.OAuth.Airtable/AirtableAuthenticationHandler.cs
Outdated
Show resolved
Hide resolved
I am going to add 2 more providers in one-two days. |
| requestMessage.Headers.Accept.Add(new MediaTypeWithQualityHeaderValue(MediaTypeNames.Application.Json)); | ||
| requestMessage.Content = new FormUrlEncodedContent(tokenRequestParameters); | ||
| requestMessage.Headers.Authorization = CreateAuthorizationHeader(); | ||
| requestMessage.Version = Backchannel.DefaultRequestVersion; |
There was a problem hiding this comment.
It's a good idea (we could also set the VersionPolicy property), but it looks like we haven't been very consistent regarding that as only 4 providers seem to set it.
@denis-goncharenko would you be interested in sending a PR to fix the inconsistency? (either by setting it for all the providers or by removing it for the 4 that currently use it, which would be less work)
There was a problem hiding this comment.
I can do it. I will set it for all providers next week in a separate PR.
This feature allows users to authenticate using their Airtable credentials.