search

Azure / azure-functions-microsoftgraph-extension

Microsoft Graph extension for Azure Functions
MIT License
46 stars 23 forks source link

Support on-behalf-of graph subscription refreshes #24

Closed ConnorMcMahon closed 6 years ago

ConnorMcMahon commented 7 years ago

In order to refresh subscription entries for all users without having an admin setup the application identity, we need to allow users to have access to the UserId and the Id attributes of the subscription that we store. This code has work to allow users to create a POCO with those fields as well as any other fields on the Subscription that they wish to access. This would allow users to write more complicated refresh calls, such as "Refresh only my mail subscriptions" or "Refresh only subscriptions that are updates".

In adding support for this, I also found a bug in cached user identity in the UserFromId mode. I simply removed the cached value.

dnfclas commented 7 years ago

@ConnorMcMahon, Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request. Thanks, .NET Foundation Pull Request Bot