-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore/414 update reload client #963
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please add a short description of the feature/fix to the Changelog.md.
Integration Test Results318 tests 318 ✅ 1m 12s ⏱️ Results for commit 6e44306. ♻️ This comment has been updated with latest results. |
Unit Test Results215 tests 215 ✅ 19s ⏱️ Results for commit 6e44306. ♻️ This comment has been updated with latest results. |
...g/eclipse/tractusx/traceability/qualitynotification/domain/base/service/HttpCallService.java
Fixed
Show fixed
Hide fixed
...g/eclipse/tractusx/traceability/qualitynotification/domain/base/service/HttpCallService.java
Fixed
Show fixed
Hide fixed
|
var body = response.getBody(); | ||
if (!response.getStatusCode().is2xxSuccessful() || body == null) { | ||
throw new BadRequestException(format("Control plane responded with: %s %s", response.getStatusCode(), body != null ? body.toString() : "")); | ||
log.info("Control plane responded with response: {}", response); |
Check failure
Code scanning / CodeQL
Log Injection
|
|
…t-information bug: #963 fix contract api
resolves eclipse-tractusx#414