Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Filters and reording of MFAreport #2162

Merged
merged 4 commits into from
Feb 24, 2024
Merged

Filters and reording of MFAreport #2162

Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
685f936
Merge pull request #8 from KelvinTegelaar/dev
kris6673 Feb 19, 2024
664d002
Merge pull request #9 from KelvinTegelaar/dev
kris6673 Feb 23, 2024
938f670
Merge branch 'KelvinTegelaar:dev' into dev
kris6673 Feb 24, 2024
32fb278
Add filters and reorder table
kris6673 Feb 24, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 20 additions & 10 deletions src/views/identity/reports/MFAReport.jsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,19 +25,20 @@ const columns = [
cell: cellBooleanFormatter({ colourless: true }),
exportSelector: 'isLicensed',
},
{
selector: (row) => row['PerUser'],
name: 'Per user MFA Status',
sortable: true,
exportSelector: 'PerUser',
},
{
selector: (row) => row['MFARegistration'],
name: 'Registered for Conditional MFA',
sortable: true,
cell: cellBooleanFormatter(),
exportSelector: 'MFARegistration',
},
{
selector: (row) => row['CoveredBySD'],
name: 'Enforced via Security Defaults',
sortable: true,
cell: cellBooleanFormatter({ colourless: true }),
exportSelector: 'CoveredBySD',
},
{
selector: (row) => row['CoveredByCA'],
name: 'Enforced via Conditional Access',
Expand All @@ -46,11 +47,10 @@ const columns = [
exportSelector: 'CoveredByCA',
},
{
selector: (row) => row['CoveredBySD'],
name: 'Enforced via Security Defaults',
selector: (row) => row['PerUser'],
name: 'Per user MFA Status',
sortable: true,
cell: cellBooleanFormatter({ colourless: true }),
exportSelector: 'CoveredBySD',
exportSelector: 'PerUser',
},
]

Expand Down Expand Up @@ -134,7 +134,17 @@ const MFAList = () => {
datatable={{
filterlist: [
{ filterName: 'Enabled users', filter: '"accountEnabled":true' },
{ filterName: 'No guest users', filter: 'Complex: UPN notlike #EXT#' },
{ filterName: 'Licensed users', filter: 'Complex: IsLicensed eq true' },
{
filterName: 'Enabled, licensed users missing MFA',
filter:
'Complex: UPN notlike #EXT#; IsLicensed eq true; accountEnabled eq true; MFARegistration eq false',
},
{
filterName: 'No MFA methods registered',
filter: 'Complex: MFARegistration eq false',
},
],
columns: tenant.defaultDomainName === 'AllTenants' ? Altcolumns : columns,
path: '/api/ListMFAUsers',
Expand Down
Loading