Skip to content

Commit

Permalink
Issue #456: Fix missing dom4j dependency in the WBEM extension
Browse files Browse the repository at this point in the history
* Corrected exception message.
* Updated shade plugin to include the missing dom4j dependency.
  • Loading branch information
NassimBtk committed Oct 17, 2024
1 parent 757172a commit f72b042
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
1 change: 1 addition & 0 deletions metricshub-wbem-extension/pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -179,6 +179,7 @@
<include>org.sentrysoftware:wbem</include>
<include>org.sentrysoftware:vcenter</include>
<include>com.vmware:vijava</include>
<include>dom4j:dom4j</include>
<include>org.sentrysoftware:metricshub-wbem-extension</include>
</includes>
<excludes>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -186,7 +186,8 @@ private String refreshVCenterTicket(
if (e instanceof InterruptedException) {
Thread.currentThread().interrupt();
}
log.error("Hostname {} - Vcenter ticket refresh query failed. Exception: {}", e);
log.error("Hostname {} - vCenter ticket refresh query failed. Error message: {}", hostname, e.getMessage());
log.debug("Hostname {} - vCenter ticket refresh query failed.", hostname, e);
throw new ClientException("vCenter refresh ticket query failed on " + hostname + ".", e);
}
}
Expand Down

0 comments on commit f72b042

Please sign in to comment.