gitgabrio commented on code in PR #4119:
URL:
https://github.com/apache/incubator-kie-kogito-runtimes/pull/4119#discussion_r2494990704
##########
addons/common/process-instance-migration/src/main/java/org/kie/kogito/process/migration/BaseProcessInstanceMigrationResource.java:
##########
@@ -50,7 +50,7 @@ public T doMigrateInstance(String processId,
ProcessMigrationSpec migrationSpec,
message.put("processInstanceId", processInstanceId);
return buildOkResponse(message);
} catch (Exception e) {
- return badRequestResponse(e.getMessage());
+ return badRequestResponse("An exception occurred during migration
of a process instance. Please check the logs for more information.");
Review Comment:
HI @baldimir
I see the logic, but if an exception is thrown, and we completely hide it
here, how could we know what happened ? Sorry, probably I miss important
details of the context
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
