This is an automated email from the ASF dual-hosted git repository. kwin pushed a commit to branch feature/ease-use-of-filterproviderimpl in repository https://gitbox.apache.org/repos/asf/jackrabbit-oak.git
The following commit(s) were added to refs/heads/feature/ease-use-of-filterproviderimpl by this push: new 0f2cc6d400 Restore nullness annotations 0f2cc6d400 is described below commit 0f2cc6d400ffde2491ce5227a0a4a00073109824 Author: Konrad Windszus <k...@apache.org> AuthorDate: Tue Jul 11 15:05:29 2023 +0200 Restore nullness annotations Remove non-related code cleanup --- .../authorization/principalbased/impl/FilterProviderImpl.java | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java b/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java index eb903887de..6eb212c7b6 100644 --- a/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java +++ b/oak-authorization-principalbased/src/main/java/org/apache/jackrabbit/oak/spi/security/authorization/principalbased/impl/FilterProviderImpl.java @@ -81,7 +81,7 @@ public class FilterProviderImpl implements FilterProvider { * @param oakPath the repository path where the principals are located * @since 1.54 */ - public FilterProviderImpl(String oakPath) { + public FilterProviderImpl(@NotNull String oakPath) { setPath(oakPath); } @@ -112,16 +112,16 @@ public class FilterProviderImpl implements FilterProvider { } @Activate - protected void activate(Configuration configuration) { + protected void activate(Configuration configuration, Map<String, Object> properties) { setPath(configuration.path()); } @Modified - protected void modified(Configuration configuration) { + protected void modified(Configuration configuration, Map<String, Object> properties) { setPath(configuration.path()); } - private void setPath(String path) { + private void setPath(@NotNull String path) { checkState(isValidPath(path), "Configured path must be a valid absolute path."); oakPath = path; }