New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

we should specify Uri, otherwise it will throw NRE #1022

Merged

jansenbe merged 1 commit into pnp:dev from dohly:hotfix/GetAccessTokenNRE

May 23, 2024

Contributor

dohly commented May 21, 2024

Repro steps (F#):

let am=new AuthenticationManager()
let ctx=am.GetACSAppOnlyContext("https://contoso.sharepoint.com/sites/mysite", clientId, clientSecret)
//..................................
let cs=ctx.GetContextSettings()
let t=cs.AuthenticationManager.GetAccessToken("https://contoso.sharepoint.com/sites/mysite")


          we should specify URL, otherwise it will throw NRE

0d01f68

jansenbe self-assigned this

jansenbe merged commit 11ad0db into pnp:dev

jansenbe added a commit that referenced this pull request


          Merge of #1022, thanks @dohly

958c881

# for free to join this conversation on GitHub. Already have an account? # to comment

Labels

None yet