Skip to content
This repository was archived by the owner on Nov 1, 2023. It is now read-only.

Use Graph SDK instead of manually constructing queries #2324

Merged
merged 2 commits into from
Sep 7, 2022
Merged

Conversation

Porges
Copy link
Member

@Porges Porges commented Aug 30, 2022

Use the Graph SDK (which we already reference) to query AD groups.

This will also cache the access token, unlike our current method. Fetching a new MSI token is quite slow (2+ seconds).

image


Needs verification in live environment.

@Porges Porges force-pushed the graph-sdk branch 4 times, most recently from 58b4f8d to 2936cf2 Compare September 4, 2022 21:22
@Porges Porges marked this pull request as ready for review September 5, 2022 00:09
@Porges Porges enabled auto-merge (squash) September 6, 2022 22:50
@Porges Porges merged commit fe51e01 into main Sep 7, 2022
@Porges Porges deleted the graph-sdk branch September 12, 2022 02:36
@ghost ghost locked as resolved and limited conversation to collaborators Oct 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants