lingwei.kong/hawkbit
1
0
Fork 0
Code Issues Pull Requests Actions 2 Packages Projects Releases Wiki Activity

Add logging for db execptons handling (#2155)

Browse Source 
Signed-off-by: Avgustin Marinov <Avgustin.Marinov@bosch.com>
This commit is contained in:
Avgustin Marinov
2024-12-19 16:37:57 +02:00
committed by GitHub
parent 52fec7c82b
commit 65ef31e702
2 changed files with 9 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
hawkbit-repository/hawkbit-repository-jpa/src/main/java/org/eclipse/hawkbit/repository/jpa/aspects/ExceptionMappingAspectHandler.java
View File

@@ -71,9 +71,15 @@ public class ExceptionMappingAspectHandler implements Ordered {
// It is a AspectJ proxy which deals with exceptions. // It is a AspectJ proxy which deals with exceptions.
@SuppressWarnings({ "squid:S00112", "squid:S1162" }) @SuppressWarnings({ "squid:S00112", "squid:S1162" })
public void catchAndWrapJpaExceptionsService(final Exception ex) throws Throwable { public void catchAndWrapJpaExceptionsService(final Exception ex) throws Throwable {
if (log.isTraceEnabled()) {
log.trace("Handling exception {}", ex.getClass().getName(), ex);
} else {
log.debug("Handling exception {}", ex.getClass().getName());
}
// Workaround for EclipseLink merge where it does not throw ConstraintViolationException directly in case of existing entity update // Workaround for EclipseLink merge where it does not throw ConstraintViolationException directly in case of existing entity update
if (ex instanceof TransactionSystemException) { if (ex instanceof TransactionSystemException transactionSystemException) {
throw replaceWithCauseIfConstraintViolationException((TransactionSystemException) ex); throw replaceWithCauseIfConstraintViolationException(transactionSystemException);
} }
for (final Class<?> mappedEx : MAPPED_EXCEPTION_ORDER) { for (final Class<?> mappedEx : MAPPED_EXCEPTION_ORDER) {

3
hawkbit-rest-core/src/main/java/org/eclipse/hawkbit/rest/RestConfiguration.java
View File

@@ -279,7 +279,7 @@ public class RestConfiguration {
private void logRequest(final HttpServletRequest request, final Exception ex) { private void logRequest(final HttpServletRequest request, final Exception ex) {
if (log.isTraceEnabled()) { if (log.isTraceEnabled()) {
log.debug("Handling exception {} of request {}", ex.getClass().getName(), request.getRequestURL(), ex); log.trace("Handling exception {} of request {}", ex.getClass().getName(), request.getRequestURL(), ex);
} else { } else {
log.debug("Handling exception {} of request {}", ex.getClass().getName(), request.getRequestURL()); log.debug("Handling exception {} of request {}", ex.getClass().getName(), request.getRequestURL());
} }
@@ -334,5 +334,4 @@ public class RestConfiguration {
return false; return false;
} }
} }
} }