OfficeDev / office-js-helpers

[ARCHIVED] A collection of helpers to simplify development of Office Add-ins & Microsoft Teams Tabs
MIT License
126 stars 56 forks source link

added pkce and fixed some bugs with authorization code grant #137

Open schmitch opened 5 years ago

schmitch commented 5 years ago

office-js had a major flaw for authorization code grant types, it didn't send the correct payload for a token exchange currently this fixes it by sending a correct application/x-www-form-urlencoded request and also made it possible to configure PKCE by utilizing CryptoJS

Edit: after that I will probably take a look at refresh tokens, and might implement them as well. So that users won't need to re-login like they need in a setup with implicit flow

msftclas commented 5 years ago

CLA assistant check
All CLA requirements met.