diff --git a/src/main/java/org/gitlab4j/api/systemhooks/SystemHookManager.java b/src/main/java/org/gitlab4j/api/systemhooks/SystemHookManager.java index 70c9dab527c76d69a5674bafb8919cfb948a915a..73d00af9b668338cc21a6e9bebe4e076d5d74b1a 100644 --- a/src/main/java/org/gitlab4j/api/systemhooks/SystemHookManager.java +++ b/src/main/java/org/gitlab4j/api/systemhooks/SystemHookManager.java @@ -9,6 +9,7 @@ import java.util.logging.Logger; import javax.servlet.http.HttpServletRequest; +import org.gitlab4j.api.GitLabApi; import org.gitlab4j.api.GitLabApiException; import org.gitlab4j.api.HookManager; import org.gitlab4j.api.utils.HttpRequestUtils; @@ -21,7 +22,7 @@ public class SystemHookManager extends HookManager { public static final String SYSTEM_HOOK_EVENT = "System Hook"; - private final static Logger LOG = Logger.getLogger(SystemHookManager.class.getName()); + private final static Logger LOGGER = GitLabApi.getLogger(); private final JacksonJson jacksonJson = new JacksonJson(); // Collection of objects listening for System Hook events. @@ -55,27 +56,27 @@ public class SystemHookManager extends HookManager { if (!isValidSecretToken(request)) { String message = "X-Gitlab-Token mismatch!"; - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } String eventName = request.getHeader("X-Gitlab-Event"); - LOG.info("handleEvent: X-Gitlab-Event=" + eventName); + LOGGER.info("handleEvent: X-Gitlab-Event=" + eventName); if (!SYSTEM_HOOK_EVENT.equals(eventName)) { String message = "Unsupported X-Gitlab-Event, event Name=" + eventName; - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } try { SystemHookEvent event; - if (LOG.isLoggable(Level.FINE)) { - LOG.fine(HttpRequestUtils.getShortRequestDump("System Hook", true, request)); + if (LOGGER.isLoggable(Level.FINE)) { + LOGGER.fine(HttpRequestUtils.getShortRequestDump("System Hook", true, request)); String postData = HttpRequestUtils.getPostDataAsString(request); - LOG.fine("Raw POST data:\n" + postData); + LOGGER.fine("Raw POST data:\n" + postData); event = jacksonJson.unmarshal(SystemHookEvent.class, postData); - LOG.fine(event.getEventName() + "\n" + jacksonJson.marshal(event) + "\n"); + LOGGER.fine(event.getEventName() + "\n" + jacksonJson.marshal(event) + "\n"); } else { InputStreamReader reader = new InputStreamReader(request.getInputStream()); event = jacksonJson.unmarshal(SystemHookEvent.class, reader); @@ -84,7 +85,7 @@ public class SystemHookManager extends HookManager { fireEvent(event); } catch (Exception e) { - LOG.warning("Error processing JSON data, exception=" + + LOGGER.warning("Error processing JSON data, exception=" + e.getClass().getSimpleName() + ", error=" + e.getMessage()); throw new GitLabApiException(e); } @@ -98,10 +99,10 @@ public class SystemHookManager extends HookManager { */ public void handleEvent(SystemHookEvent event) throws GitLabApiException { if (event != null) { - LOG.info("handleEvent:" + event.getClass().getSimpleName() + ", eventName=" + event.getEventName()); + LOGGER.info("handleEvent:" + event.getClass().getSimpleName() + ", eventName=" + event.getEventName()); fireEvent(event); } else { - LOG.warning("handleEvent: provided event cannot be null!"); + LOGGER.warning("handleEvent: provided event cannot be null!"); } } @@ -154,7 +155,7 @@ public class SystemHookManager extends HookManager { fireRepositoryEvent((RepositorySystemHookEvent) event); } else { String message = "Unsupported event, event_named=" + event.getEventName(); - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } } diff --git a/src/main/java/org/gitlab4j/api/webhook/WebHookManager.java b/src/main/java/org/gitlab4j/api/webhook/WebHookManager.java index bd855577d6afb1e1f3e73228ad6e962669f11cf8..174be3c9e6c24f2c571f85940ca80f32a4818a71 100644 --- a/src/main/java/org/gitlab4j/api/webhook/WebHookManager.java +++ b/src/main/java/org/gitlab4j/api/webhook/WebHookManager.java @@ -9,6 +9,7 @@ import java.util.logging.Logger; import javax.servlet.http.HttpServletRequest; +import org.gitlab4j.api.GitLabApi; import org.gitlab4j.api.GitLabApiException; import org.gitlab4j.api.HookManager; import org.gitlab4j.api.utils.HttpRequestUtils; @@ -19,7 +20,7 @@ import org.gitlab4j.api.utils.JacksonJson; */ public class WebHookManager extends HookManager { - private final static Logger LOG = Logger.getLogger(WebHookManager.class.getName()); + private final static Logger LOGGER = GitLabApi.getLogger(); private final JacksonJson jacksonJson = new JacksonJson(); // Collection of objects listening for WebHook events. @@ -53,12 +54,12 @@ public class WebHookManager extends HookManager { if (!isValidSecretToken(request)) { String message = "X-Gitlab-Token mismatch!"; - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } String eventName = request.getHeader("X-Gitlab-Event"); - LOG.info("handleEvent: X-Gitlab-Event=" + eventName); + LOGGER.info("handleEvent: X-Gitlab-Event=" + eventName); switch (eventName) { case BuildEvent.BUILD_HOOK_X_GITLAB_EVENT: @@ -74,19 +75,19 @@ public class WebHookManager extends HookManager { default: String message = "Unsupported X-Gitlab-Event, event Name=" + eventName; - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } try { Event event; - if (LOG.isLoggable(Level.FINE)) { - LOG.fine(HttpRequestUtils.getShortRequestDump(eventName + " webhook", true, request)); + if (LOGGER.isLoggable(Level.FINE)) { + LOGGER.fine(HttpRequestUtils.getShortRequestDump(eventName + " webhook", true, request)); String postData = HttpRequestUtils.getPostDataAsString(request); - LOG.fine("Raw POST data:\n" + postData); + LOGGER.fine("Raw POST data:\n" + postData); event = jacksonJson.unmarshal(Event.class, postData); - LOG.fine(event.getObjectKind() + " event:\n" + jacksonJson.marshal(event) + "\n"); + LOGGER.fine(event.getObjectKind() + " event:\n" + jacksonJson.marshal(event) + "\n"); } else { InputStreamReader reader = new InputStreamReader(request.getInputStream()); event = jacksonJson.unmarshal(Event.class, reader); @@ -95,7 +96,7 @@ public class WebHookManager extends HookManager { fireEvent(event); } catch (Exception e) { - LOG.warning("Error parsing JSON data, exception=" + e.getClass().getSimpleName() + ", error=" + e.getMessage()); + LOGGER.warning("Error parsing JSON data, exception=" + e.getClass().getSimpleName() + ", error=" + e.getMessage()); throw new GitLabApiException(e); } } @@ -108,7 +109,7 @@ public class WebHookManager extends HookManager { */ public void handleEvent(Event event) throws GitLabApiException { - LOG.info("handleEvent: object_kind=" + event.getObjectKind()); + LOGGER.info("handleEvent: object_kind=" + event.getObjectKind()); switch (event.getObjectKind()) { case BuildEvent.OBJECT_KIND: @@ -123,7 +124,7 @@ public class WebHookManager extends HookManager { default: String message = "Unsupported event object_kind, object_kind=" + event.getObjectKind(); - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } @@ -194,7 +195,7 @@ public class WebHookManager extends HookManager { default: String message = "Unsupported event object_kind, object_kind=" + event.getObjectKind(); - LOG.warning(message); + LOGGER.warning(message); throw new GitLabApiException(message); } }