Skip to content

Commit a4cfb0b

Browse files
Discookiebruntib
authored andcommitted
Remove OAuth groups source requirements
These two were a needless restriction on where groups can come from, and it prevents CodeChecker from parsing some native Entra groups.
1 parent a803267 commit a4cfb0b

1 file changed

Lines changed: 1 addition & 3 deletions

File tree

web/server/codechecker_server/api/authentication.py

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -441,9 +441,7 @@ def performLogin(self, auth_method, auth_string):
441441
morePages = True
442442
while morePages:
443443
for group in response["value"]:
444-
if group.get("onPremisesSyncEnabled") and \
445-
group.get("securityEnabled"):
446-
groups.append(group["displayName"])
444+
groups.append(group["displayName"])
447445
# paging of groups
448446
morePages = "@odata.nextLink" in response
449447
if morePages:

0 commit comments

Comments
 (0)