microsoft / microsoft/ALAppExtensions
Use OAuthAuthorityUrl Parameter in OAuth2Impl.AcquireTokenByAuthorizationCodeWithCredentials or give us an event to use it
Nobody has claimed this yet.
- Dominant language
- AL
- Stars
- 988
- Forks
- 692
- Avg merge
- 49m
- Merged PRs (30d)
- 1
Description
Why do you need this change?
Hello,
Then we can use the implementation with other oAuth provider.
Describe the request
Please Add TockenUrl as Parameter Parameter in OAuth2Impl.AcquireTokenByAuthorizationCodeWithCredentials or give us an event to use it.
procedure AcquireTokenByAuthorizationCodeWithCredentials(AuthorizationCode: Text; ClientId: Text; ClientSecret: SecretText; RedirectUrl: Text; TockenUrl : Text; Scopes: List of [Text]; var AccessToken: SecretText)
var
ScopesArray: DotNet StringArray;
begin
FillScopesArray(Scopes, ScopesArray);
OnBeforeALAcquireTokenByAuthorizationCodeWithCredentials(AuthorizationCode; ClientId; ClientSecret; RedirectUrlt; OAuthAuthorityUrl; Scopes; var AccessToken, handled);
if not handled then
AccessToken := AuthFlow.ALAcquireTokenByAuthorizationCodeWithCredentials(AuthorizationCode, ClientId, ClientSecret, ScopesArray);
end;
Internal work item: AB#610223
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start at OAuth2Impl.AcquireTokenByAuthorizationCodeWithCredentials and review how OAuthAuthorityUrl is handled in the shown flow. Determine whether completion means accepting a token URL parameter or exposing an event for alternate OAuth providers, then verify the selected behavior supports a non-default provider.
Written by the indexing model from the issue text.
Assessment
- Domain
- authentication
- Issue type
- Feature
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100