Skip to content

Removed "Limited Access" From Reports #311

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

Merged
Gunjan Datta (gudatta) merged 1 commit into from
Jul 28, 2026
Merged

Removed "Limited Access" From Reports #311

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
39 changes: 37 additions & 2 deletions src/reports/viewPermissions.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,12 @@ export class ViewPermissions {
for (let i = 0; i < roles.length; i++) {
let role = roles[i];

// Skip permissions that are "Limited Access System Group"
if (role.Member.Title.indexOf("Limited Access System Group") === 0) {
// Skip this role
continue;
}

// See if this is the eeeu, everyone or the custom group
if (role.Member.Title == "Everyone except external users" || role.Member.Title == "Everyone" || this.OversharedGroups.indexOf(role.Member.Title) >= 0) {
// Append the group
Expand Down Expand Up @@ -59,6 +65,23 @@ export class ViewPermissions {
for (let i = 0; i < roles.length; i++) {
let role = roles[i];

// Parse the role definitions
let roleDefs = role.RoleDefinitionBindings?.results || [];
let isLimitedAccess = false;
for (let j = 0; j < roleDefs.length; j++) {
if (roleDefs[j].Name === "Limited Access") {
// Skip this role
isLimitedAccess = true;
break;
}
}

// Skip permissions that are "Limited Access System Group"
if (isLimitedAccess || role.Member.Title.indexOf("Limited Access System Group") === 0) {
// Skip this role
continue;
}

// See if this is the eeeu or everyone
if (role.Member.Title == "Everyone except external users" || role.Member.Title == "Everyone" || this.OversharedGroups.indexOf(role.Member.Title) >= 0) {
// Set the flag
Expand All @@ -69,8 +92,8 @@ export class ViewPermissions {
else {
// Parse the users
let users: Types.SP.User[] = role.Member["Users"] ? role.Member["Users"].results : [];
for (let i = 0; i < users.length; i++) {
let user = users[i];
for (let j = 0; j < users.length; j++) {
let user = users[j];

// See if this is the eeeu or everyone
if (user.Title == "Everyone except external users" || user.Title == "Everyone" || this.OversharedGroups.indexOf(user.Title) >= 0) {
Expand Down Expand Up @@ -216,12 +239,24 @@ export class ViewPermissions {

// Parse the permissions
item.Permissions.forEach(role => {
let isLimitedAccess = false;

// Skip permissions that are "Limited Access System Group"
if (role.Member.Title.indexOf("Limited Access System Group") === 0) { isLimitedAccess = true; }

// Parse the role definitions
let roleDefinitions = [];
role.RoleDefinitionBindings.results.forEach(roleDef => {
// Skip limited access permissions
if (roleDef.Name.indexOf("Limited Access") === 0) { isLimitedAccess = true; return; }

// Append the permission
roleDefinitions.push(`<span><b>${roleDef.Name}:</b> ${roleDef.Description}${roleDef.Hidden ? " (Hidden)" : ""}</span>`);
});

// Skip this role if it is limited access
if (isLimitedAccess) { return; }

// See if this is a user
switch (role.Member.PrincipalType) {
case SPTypes.PrincipalTypes.User:
Expand Down
Loading