oEscal commented on PR #3937: URL: https://github.com/apache/incubator-kie-kogito-runtimes/pull/3937#issuecomment-3348051045
> @oEscal You need to take into account [PR](https://github.com/apache/incubator-kie-kogito-runtimes/pull/4014) which I bet you did not know it exist. > > This PR added support for $WORKFLOW.statusCode and $WORKFLOW.statusMessage. > > So, since there is already a way to access error status and error message, we just need to add access to response headers following a similar approach; by specifying metadata in the workflow definition file, , the response headers are made accesible through new $WORKFLOW context variable (rather than including them in the model directly) > > Unfortunately, This implies basically reimplementing the PR. RestWorkItemHandler and RestTypeHandler does not need to be modified, but JsonNodeResultHandler to check the new metadata and RestKogitoProcessContextResolver to add the context. Once you have that, you add a IT test with a new WorkflowDefinition using that metadata. Sorry about this and thanks for the efffort! @fjtirado yes, I later noticed that PR when fixing merge conflicts (because this PR was older). However, I think this PR is a bit different. Because the idea is not to access the headers or status code of the overall workflow, but instead to access the previously running function, so that the workflow and its actions and functions can use that data in their logic. -- 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]
