SpecterOps / BloodHound

Six Degrees of Domain Admin
https://bloodhoundenterprise.io/
Apache License 2.0
1.14k stars 113 forks source link

fix: include appropriate oidcs scopes + misc refactor #939

Closed ddlees closed 1 week ago

ddlees commented 1 week ago

Description

Describe your changes in detail

When testing against Auth0 we discovered that the id_token was not being returned as expected. To remedy this we explicitly declare all the requires scopes in this PR.

Motivation and Context

This PR addresses: BED-4768

Why is this change required? What problem does it solve?

How Has This Been Tested?

Please describe in detail how you tested your changes. Include details of your testing environment, and the tests you ran to see how your change affects other areas of the code, etc.

I have updated all the tests to account for my change.

Screenshots (optional):

Types of changes

Checklist: