From fc2b6a40c299c7b6f98eebce40439f969e23323c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Aur=C3=A9lien=20Manteaux?= Date: Wed, 18 Sep 2024 15:37:20 +0200 Subject: [PATCH] Clean up loggers, guava usage and instance of usages --- .../coreoz/plume/conf/RequiredIncluder.java | 14 +++++----- .../coreoz/plume/mail/MailerProviderTest.java | 10 +++---- .../healthchecks/DatabaseHealthCheck.java | 5 ++-- .../utils/metrics/MetricsCheckBuilder.java | 4 +-- .../plume/jersey/async/AsyncJersey.java | 23 ++++++--------- .../plume/jersey/errors/Validators.java | 13 +++++---- .../plume/jersey/errors/WsException.java | 8 ++---- .../jersey/errors/WsJacksonJsonProvider.java | 22 ++++++--------- .../errors/WsResultExceptionMapper.java | 28 ++++++++----------- .../grizzly/GrizzlyErrorPageHandler.java | 8 ++---- .../plume/jersey/java8/InstantConverter.java | 16 ++++------- .../jersey/java8/LocalDateConverter.java | 14 +++------- .../jersey/java8/LocalDateTimeConverter.java | 14 +++------- .../security/basic/BasicAuthenticator.java | 15 ++++------ .../permission/PermissionFeature.java | 8 ++---- .../RequireExplicitAccessControlFeature.java | 20 +++++-------- 16 files changed, 81 insertions(+), 141 deletions(-) diff --git a/plume-conf/src/main/java/com/coreoz/plume/conf/RequiredIncluder.java b/plume-conf/src/main/java/com/coreoz/plume/conf/RequiredIncluder.java index aa9cebe..d9a7e15 100644 --- a/plume-conf/src/main/java/com/coreoz/plume/conf/RequiredIncluder.java +++ b/plume-conf/src/main/java/com/coreoz/plume/conf/RequiredIncluder.java @@ -14,7 +14,7 @@ /** * Config includer that forces all includes to resolve or else fail with a ConfigException. - * + * * See https://github.com/lightbend/config/issues/587 */ public class RequiredIncluder implements ConfigIncluder, ConfigIncluderFile, ConfigIncluderURL, @@ -42,8 +42,8 @@ public ConfigObject include(ConfigIncludeContext context, String what) { @Override public ConfigObject includeResources(ConfigIncludeContext context, String what) { context = verifyDelegateAndStrictContext(context); - if (delegate instanceof ConfigIncluderClasspath) { - return ((ConfigIncluderClasspath) delegate).includeResources(context, what); + if (delegate instanceof ConfigIncluderClasspath configIncluderClasspath) { + return configIncluderClasspath.includeResources(context, what); } return ConfigFactory.parseResources(what, context.parseOptions()).root(); } @@ -51,8 +51,8 @@ public ConfigObject includeResources(ConfigIncludeContext context, String what) @Override public ConfigObject includeFile(ConfigIncludeContext context, File what) { context = verifyDelegateAndStrictContext(context); - if (delegate instanceof ConfigIncluderFile) { - return ((ConfigIncluderFile) delegate).includeFile(context, what); + if (delegate instanceof ConfigIncluderFile configIncluderFile) { + return configIncluderFile.includeFile(context, what); } return ConfigFactory.parseFile(what, context.parseOptions()).root(); } @@ -60,8 +60,8 @@ public ConfigObject includeFile(ConfigIncludeContext context, File what) { @Override public ConfigObject includeURL(ConfigIncludeContext context, URL what) { context = verifyDelegateAndStrictContext(context); - if (delegate instanceof ConfigIncluderURL) { - return ((ConfigIncluderURL) delegate).includeURL(context, what); + if (delegate instanceof ConfigIncluderURL configIncluderUrl) { + return configIncluderUrl.includeURL(context, what); } return ConfigFactory.parseURL(what, context.parseOptions()).root(); } diff --git a/plume-mail/src/test/java/com/coreoz/plume/mail/MailerProviderTest.java b/plume-mail/src/test/java/com/coreoz/plume/mail/MailerProviderTest.java index cbc71c7..9663f73 100644 --- a/plume-mail/src/test/java/com/coreoz/plume/mail/MailerProviderTest.java +++ b/plume-mail/src/test/java/com/coreoz/plume/mail/MailerProviderTest.java @@ -1,17 +1,16 @@ package com.coreoz.plume.mail; +import com.typesafe.config.Config; +import com.typesafe.config.ConfigFactory; import org.assertj.core.api.Assertions; import org.junit.Test; -import com.google.common.collect.ImmutableMap; -import com.typesafe.config.Config; -import com.typesafe.config.ConfigFactory; +import java.util.Map; public class MailerProviderTest { - @Test public void should_generate_well_form_property_file() { - Config config = ConfigFactory.parseMap(ImmutableMap.of( + Config config = ConfigFactory.parseMap(Map.of( "mail.\"javaxmail.debug\"", "true", "mail.transportstrategy", "SMTP_SSL" )); @@ -22,5 +21,4 @@ public void should_generate_well_form_property_file() { + "simplejavamail.transportstrategy=SMTP_SSL" ); } - } diff --git a/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/health/healthchecks/DatabaseHealthCheck.java b/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/health/healthchecks/DatabaseHealthCheck.java index 3bcaa15..425e7b6 100644 --- a/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/health/healthchecks/DatabaseHealthCheck.java +++ b/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/health/healthchecks/DatabaseHealthCheck.java @@ -2,13 +2,12 @@ import com.codahale.metrics.health.HealthCheck; import com.coreoz.plume.db.transaction.TransactionManager; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import lombok.extern.slf4j.Slf4j; import java.sql.SQLException; +@Slf4j public class DatabaseHealthCheck extends HealthCheck { - private static final Logger logger = LoggerFactory.getLogger(DatabaseHealthCheck.class); private final TransactionManager transactionManager; public DatabaseHealthCheck(TransactionManager transactionManager) { diff --git a/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/metrics/MetricsCheckBuilder.java b/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/metrics/MetricsCheckBuilder.java index 88db257..1c3d6d0 100644 --- a/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/metrics/MetricsCheckBuilder.java +++ b/plume-web-jersey-monitoring/src/main/java/com/coreoz/plume/jersey/monitoring/utils/metrics/MetricsCheckBuilder.java @@ -13,8 +13,8 @@ public class MetricsCheckBuilder { private final MetricRegistry metricRegistry = new MetricRegistry(); public MetricsCheckBuilder registerMetric(String name, Metric metric) { - if (metric instanceof MetricSet) { - this.metricRegistry.registerAll(name, (MetricSet) metric); + if (metric instanceof MetricSet metricSet) { + this.metricRegistry.registerAll(name, metricSet); } else { this.metricRegistry.register(name, metric); } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/async/AsyncJersey.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/async/AsyncJersey.java index 777d914..d58643e 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/async/AsyncJersey.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/async/AsyncJersey.java @@ -1,26 +1,21 @@ package com.coreoz.plume.jersey.async; -import java.util.concurrent.CompletableFuture; -import java.util.function.BiConsumer; - -import javax.ws.rs.container.AsyncResponse; - -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - import com.coreoz.plume.jersey.errors.ErrorResponse; import com.coreoz.plume.jersey.errors.WsError; -import com.google.common.collect.ImmutableList; +import lombok.extern.slf4j.Slf4j; + +import javax.ws.rs.container.AsyncResponse; +import java.util.List; +import java.util.concurrent.CompletableFuture; +import java.util.function.BiConsumer; /** * Provides a bridge between JAX-RS asynchronous API and Java 8 asynchronous API * @see CompletableFuture * @see AsyncResponse */ +@Slf4j public class AsyncJersey { - - private static final Logger logger = LoggerFactory.getLogger(AsyncJersey.class); - /** * Provides a {@link BiConsumer} from an {@link AsyncResponse} * that should be called when a {@link CompletableFuture} is terminated.
@@ -41,10 +36,8 @@ public class AsyncJersey { asyncResponse.resume(responseBody); } else { logger.error("An exception was raised during the promise execution", exception); - asyncResponse.resume(new ErrorResponse(WsError.INTERNAL_ERROR, ImmutableList.of())); + asyncResponse.resume(new ErrorResponse(WsError.INTERNAL_ERROR, List.of())); } }; } - } - diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/Validators.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/Validators.java index c5c1f88..cbb3280 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/Validators.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/Validators.java @@ -3,10 +3,11 @@ import org.apache.commons.validator.routines.EmailValidator; import com.google.common.base.Strings; -import com.google.common.collect.ImmutableList; import org.jetbrains.annotations.NotNull; import org.jetbrains.annotations.Nullable; +import java.util.List; + /** * Provides common validators that will throw {@link WsException} * when the input data does not pass the validator. @@ -21,28 +22,28 @@ public class Validators { public static @NotNull String checkRequired(@NotNull String fieldName, @Nullable String fieldValue) { if(Strings.isNullOrEmpty(fieldValue)) { - throw new WsException(WsError.FIELD_REQUIRED, ImmutableList.of(fieldName)); + throw new WsException(WsError.FIELD_REQUIRED, List.of(fieldName)); } return fieldValue; } public static @NotNull T checkRequired(@NotNull String fieldName, @Nullable T fieldValue) { if(fieldValue == null) { - throw new WsException(WsError.FIELD_REQUIRED, ImmutableList.of(fieldName)); + throw new WsException(WsError.FIELD_REQUIRED, List.of(fieldName)); } return fieldValue; } public static @Nullable String checkEmail(@NotNull String fieldName, @Nullable String fieldValue) { if(!EmailValidator.getInstance().isValid(fieldValue)) { - throw new WsException(WsError.EMAIL_INVALID, ImmutableList.of(fieldName)); + throw new WsException(WsError.EMAIL_INVALID, List.of(fieldName)); } return fieldValue; } public static @Nullable String checkHexaColor(@NotNull String fieldName, @Nullable String fieldValue) { if(fieldValue != null && !fieldValue.matches("[0-9a-fA-F]{6}")) { - throw new WsException(WsError.COLOR_INVALID, ImmutableList.of(fieldName)); + throw new WsException(WsError.COLOR_INVALID, List.of(fieldName)); } return fieldValue; } @@ -50,7 +51,7 @@ public class Validators { public static @Nullable String checkHexaColorWithStartingHash(@NotNull String fieldName, @Nullable String fieldValue) { if(fieldValue != null) { if (!fieldValue.startsWith("#")) { - throw new WsException(WsError.COLOR_INVALID, ImmutableList.of(fieldName)); + throw new WsException(WsError.COLOR_INVALID, List.of(fieldName)); } return "#" + checkHexaColor(fieldName, fieldValue.substring(1)); } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsException.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsException.java index 769af87..5d48c50 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsException.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsException.java @@ -1,8 +1,6 @@ package com.coreoz.plume.jersey.errors; -import com.google.common.collect.ImmutableList; - -import java.io.Serial; +import java.util.List; /** * A {@link RuntimeException} that stops the execution of the web-service @@ -16,11 +14,11 @@ public class WsException extends RuntimeException { private final Iterable statusArguments; public WsException(WsError error) { - this(error, ImmutableList.of()); + this(error, List.of()); } public WsException(WsError error, String... statusArguments) { - this(error, ImmutableList.copyOf(statusArguments)); + this(error, List.of(statusArguments)); } public WsException(WsError error, Iterable statusArguments) { diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsJacksonJsonProvider.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsJacksonJsonProvider.java index fc709ef..36657d5 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsJacksonJsonProvider.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsJacksonJsonProvider.java @@ -1,30 +1,25 @@ package com.coreoz.plume.jersey.errors; +import com.fasterxml.jackson.databind.ObjectMapper; +import com.fasterxml.jackson.jaxrs.cfg.Annotations; +import com.fasterxml.jackson.jaxrs.json.JacksonJaxbJsonProvider; +import lombok.extern.slf4j.Slf4j; + +import javax.ws.rs.core.MediaType; +import javax.ws.rs.core.MultivaluedMap; import java.io.IOException; import java.io.InputStream; import java.lang.annotation.Annotation; import java.lang.reflect.Type; -import javax.ws.rs.core.MediaType; -import javax.ws.rs.core.MultivaluedMap; - -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.fasterxml.jackson.databind.ObjectMapper; -import com.fasterxml.jackson.jaxrs.cfg.Annotations; -import com.fasterxml.jackson.jaxrs.json.JacksonJaxbJsonProvider; - /** * A Jackson JSON provider that throws a {@link JsonRequestParseException} * during the parsing. * It is useful to return proper 400 errors when JSON request input is not valid * @see JacksonJaxbJsonProvider */ +@Slf4j public class WsJacksonJsonProvider extends JacksonJaxbJsonProvider { - - private static final Logger logger = LoggerFactory.getLogger(WsJacksonJsonProvider.class); - public WsJacksonJsonProvider() { super(); } @@ -48,5 +43,4 @@ public Object readFrom(Class type, Type genericType, Annotation[] annota throw new JsonRequestParseException(); } } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsResultExceptionMapper.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsResultExceptionMapper.java index db37ae9..34babae 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsResultExceptionMapper.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/errors/WsResultExceptionMapper.java @@ -1,39 +1,34 @@ package com.coreoz.plume.jersey.errors; +import lombok.extern.slf4j.Slf4j; + import javax.ws.rs.WebApplicationException; import javax.ws.rs.core.MediaType; import javax.ws.rs.core.Response; import javax.ws.rs.core.Response.Status; import javax.ws.rs.ext.ExceptionMapper; import javax.ws.rs.ext.Provider; +import java.util.List; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.google.common.collect.ImmutableList; - +@Slf4j @Provider public class WsResultExceptionMapper implements ExceptionMapper { - - private static final Logger logger = LoggerFactory.getLogger(WsResultExceptionMapper.class); - @Override public Response toResponse(Throwable e) { - if (e instanceof WsException) { - WsException wsException = (WsException) e; - return Response + if (e instanceof WsException wsException) { + return Response .status(Status.BAD_REQUEST) .entity(new ErrorResponse(wsException.getError(), wsException.getStatusArguments())) .type(MediaType.APPLICATION_JSON_TYPE) .build(); } - if(e instanceof WebApplicationException) { - return ((WebApplicationException) e).getResponse(); + if(e instanceof WebApplicationException webApplicationException) { + return webApplicationException.getResponse(); } if(e instanceof JsonRequestParseException) { return Response .status(Status.BAD_REQUEST) - .entity(new ErrorResponse(WsError.REQUEST_INVALID, ImmutableList.of("JSON object supplied in request is invalid"))) + .entity(new ErrorResponse(WsError.REQUEST_INVALID, List.of("JSON object supplied in request is invalid"))) .type(MediaType.APPLICATION_JSON_TYPE) .build(); } @@ -42,9 +37,8 @@ public Response toResponse(Throwable e) { return Response .status(Status.INTERNAL_SERVER_ERROR) - .entity(new ErrorResponse(WsError.INTERNAL_ERROR, ImmutableList.of())) + .entity(new ErrorResponse(WsError.INTERNAL_ERROR, List.of())) .type(MediaType.APPLICATION_JSON_TYPE) .build(); } - -} \ No newline at end of file +} diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/grizzly/GrizzlyErrorPageHandler.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/grizzly/GrizzlyErrorPageHandler.java index f928353..ddbc47a 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/grizzly/GrizzlyErrorPageHandler.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/grizzly/GrizzlyErrorPageHandler.java @@ -1,17 +1,14 @@ package com.coreoz.plume.jersey.grizzly; +import lombok.extern.slf4j.Slf4j; import org.glassfish.grizzly.http.server.ErrorPageGenerator; import org.glassfish.grizzly.http.server.Request; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; /** * Replace the default Grizzly error handler to avoid leaking server information */ +@Slf4j public class GrizzlyErrorPageHandler implements ErrorPageGenerator { - - private static final Logger logger = LoggerFactory.getLogger(GrizzlyErrorPageHandler.class); - @Override public String generate(Request request, int status, String reasonPhrase, String description, Throwable exception) { @@ -27,5 +24,4 @@ public String generate(Request request, int status, String reasonPhrase, String return status + " " + reasonPhrase; } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/InstantConverter.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/InstantConverter.java index 536ed55..8a90c0d 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/InstantConverter.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/InstantConverter.java @@ -1,20 +1,15 @@ package com.coreoz.plume.jersey.java8; +import com.google.common.base.Strings; +import lombok.extern.slf4j.Slf4j; + +import javax.ws.rs.ext.ParamConverter; import java.time.Instant; import java.time.LocalDateTime; import java.time.ZoneId; -import javax.ws.rs.ext.ParamConverter; - -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.google.common.base.Strings; - +@Slf4j public class InstantConverter implements ParamConverter { - - private static final Logger logger = LoggerFactory.getLogger(InstantConverter.class); - @Override public Instant fromString(String value) { try { @@ -37,5 +32,4 @@ public Instant fromString(String value) { public String toString(Instant value) { return value == null ? null : value.toString(); } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateConverter.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateConverter.java index c839481..c98a5b5 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateConverter.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateConverter.java @@ -1,18 +1,13 @@ package com.coreoz.plume.jersey.java8; -import java.time.LocalDate; +import com.google.common.base.Strings; +import lombok.extern.slf4j.Slf4j; import javax.ws.rs.ext.ParamConverter; +import java.time.LocalDate; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.google.common.base.Strings; - +@Slf4j public class LocalDateConverter implements ParamConverter { - - private static final Logger logger = LoggerFactory.getLogger(LocalDateConverter.class); - @Override public LocalDate fromString(String value) { try { @@ -27,5 +22,4 @@ public LocalDate fromString(String value) { public String toString(LocalDate value) { return value == null ? null : value.toString(); } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateTimeConverter.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateTimeConverter.java index 173e500..afc7e02 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateTimeConverter.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/java8/LocalDateTimeConverter.java @@ -1,18 +1,13 @@ package com.coreoz.plume.jersey.java8; -import java.time.LocalDateTime; +import com.google.common.base.Strings; +import lombok.extern.slf4j.Slf4j; import javax.ws.rs.ext.ParamConverter; +import java.time.LocalDateTime; -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.google.common.base.Strings; - +@Slf4j public class LocalDateTimeConverter implements ParamConverter { - - private static final Logger logger = LoggerFactory.getLogger(LocalDateTimeConverter.class); - @Override public LocalDateTime fromString(String value) { try { @@ -27,5 +22,4 @@ public LocalDateTime fromString(String value) { public String toString(LocalDateTime value) { return value == null ? null : value.toString(); } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/basic/BasicAuthenticator.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/basic/BasicAuthenticator.java index a28fb88..2789a36 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/basic/BasicAuthenticator.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/basic/BasicAuthenticator.java @@ -1,8 +1,6 @@ package com.coreoz.plume.jersey.security.basic; -import java.nio.charset.StandardCharsets; -import java.util.Base64; -import java.util.function.Function; +import lombok.extern.slf4j.Slf4j; import javax.ws.rs.ClientErrorException; import javax.ws.rs.ForbiddenException; @@ -10,19 +8,17 @@ import javax.ws.rs.core.HttpHeaders; import javax.ws.rs.core.Response; import javax.ws.rs.core.Response.Status; - -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; +import java.nio.charset.StandardCharsets; +import java.util.Base64; +import java.util.function.Function; /** * Enable to easily manage HTTP Basic Authentication with Jersey. * * @param The user type handled by the authentication */ +@Slf4j public class BasicAuthenticator { - - private static final Logger logger = LoggerFactory.getLogger(BasicAuthenticator.class); - private static final String BASIC_PREFIX = "Basic "; private final Function authenticator; @@ -136,5 +132,4 @@ public static Credentials parseBasicHeader(String authorizationHeader) { return new Credentials(decodedCredentials[0], decodedCredentials[1]); } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/PermissionFeature.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/PermissionFeature.java index 74b447f..d1f6b76 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/PermissionFeature.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/PermissionFeature.java @@ -13,6 +13,7 @@ import javax.ws.rs.container.ResourceInfo; import javax.ws.rs.core.FeatureContext; +import lombok.extern.slf4j.Slf4j; import org.glassfish.jersey.server.internal.LocalizationMessages; import org.slf4j.Logger; import org.slf4j.LoggerFactory; @@ -22,10 +23,8 @@ * * @param The annotation type from which a permission will be fetched */ +@Slf4j public class PermissionFeature implements DynamicFeature { - - private static final Logger logger = LoggerFactory.getLogger(PermissionFeature.class); - private final PermissionRequestProvider requestPermissionProvider; private final Class permissionAnnotationType; private final Function permissionAnnotationExtractor; @@ -73,7 +72,6 @@ private boolean addPermissionFilter(AnnotatedElement annotatedElement, FeatureCo } private static class PermissionRequestFilter implements ContainerRequestFilter { - private final String resourcePermission; private final PermissionRequestProvider requestPermissionProvider; @@ -106,7 +104,5 @@ private boolean authorize(ContainerRequestContext requestContext, String permiss return isAuthorized; } - } - } diff --git a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/RequireExplicitAccessControlFeature.java b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/RequireExplicitAccessControlFeature.java index 2860cf6..740d535 100644 --- a/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/RequireExplicitAccessControlFeature.java +++ b/plume-web-jersey/src/main/java/com/coreoz/plume/jersey/security/permission/RequireExplicitAccessControlFeature.java @@ -1,9 +1,7 @@ package com.coreoz.plume.jersey.security.permission; -import java.io.IOException; -import java.lang.annotation.Annotation; -import java.lang.reflect.AnnotatedElement; -import java.util.Set; +import com.google.common.collect.ImmutableSet; +import lombok.extern.slf4j.Slf4j; import javax.ws.rs.ForbiddenException; import javax.ws.rs.container.ContainerRequestContext; @@ -11,11 +9,10 @@ import javax.ws.rs.container.DynamicFeature; import javax.ws.rs.container.ResourceInfo; import javax.ws.rs.core.FeatureContext; - -import org.slf4j.Logger; -import org.slf4j.LoggerFactory; - -import com.google.common.collect.ImmutableSet; +import java.io.IOException; +import java.lang.annotation.Annotation; +import java.lang.reflect.AnnotatedElement; +import java.util.Set; /** * Force explicit access control being added on each exposed API. @@ -27,10 +24,8 @@ * Usage when configuring Jersey is:
* {@code resourceConfig.register(RequireExplicitAccessControlFeature.accessControlAnnotations(PublicApi.class, RestrictToAdmin.class));} */ +@Slf4j public class RequireExplicitAccessControlFeature implements DynamicFeature { - - private static final Logger logger = LoggerFactory.getLogger(RequireExplicitAccessControlFeature.ForbiddenAccessFilter.class); - private final Set> registeredAccessControlAnnotations; public RequireExplicitAccessControlFeature(Set> registeredAccessControlAnnotations) { @@ -78,5 +73,4 @@ public void filter(ContainerRequestContext requestContext) throws IOException { throw new ForbiddenException(); } } - }