Make allAuthorities unmodifiable (#2551)
Signed-off-by: Avgustin Marinov <Avgustin.Marinov@bosch.com>
This commit is contained in:
@@ -12,6 +12,7 @@ package org.eclipse.hawkbit.repository.test.util;
|
|||||||
import java.io.Serial;
|
import java.io.Serial;
|
||||||
import java.io.Serializable;
|
import java.io.Serializable;
|
||||||
import java.lang.annotation.Annotation;
|
import java.lang.annotation.Annotation;
|
||||||
|
import java.util.ArrayList;
|
||||||
import java.util.Arrays;
|
import java.util.Arrays;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
import java.util.Objects;
|
import java.util.Objects;
|
||||||
@@ -164,7 +165,7 @@ public class SecurityContextSwitch {
|
|||||||
}
|
}
|
||||||
|
|
||||||
private String[] getAllAuthorities(final String[] additionalAuthorities, final String[] notInclude) {
|
private String[] getAllAuthorities(final String[] additionalAuthorities, final String[] notInclude) {
|
||||||
final List<String> permissions = SpPermission.getAllAuthorities();
|
final List<String> permissions = new ArrayList<>(SpPermission.getAllAuthorities()); // list is unmodifiable
|
||||||
if (notInclude != null) {
|
if (notInclude != null) {
|
||||||
permissions.removeAll(Arrays.asList(notInclude));
|
permissions.removeAll(Arrays.asList(notInclude));
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -12,6 +12,7 @@ package org.eclipse.hawkbit.im.authentication;
|
|||||||
import java.lang.reflect.Field;
|
import java.lang.reflect.Field;
|
||||||
import java.lang.reflect.Modifier;
|
import java.lang.reflect.Modifier;
|
||||||
import java.util.ArrayList;
|
import java.util.ArrayList;
|
||||||
|
import java.util.Collections;
|
||||||
import java.util.List;
|
import java.util.List;
|
||||||
|
|
||||||
import lombok.AccessLevel;
|
import lombok.AccessLevel;
|
||||||
@@ -168,7 +169,7 @@ public final class SpPermission {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
return allPermissions;
|
return Collections.unmodifiableList(allPermissions);
|
||||||
});
|
});
|
||||||
/**
|
/**
|
||||||
* Return all permission.
|
* Return all permission.
|
||||||
|
|||||||
Reference in New Issue
Block a user