Skip to content

[incubator-kie#7147] NullPointerException in embedded job service whe… - #7148

Open
abhijithumbe wants to merge 2 commits into
apache:mainfrom
abhijithumbe:kie-7147
Open

abhijithumbe wants to merge 2 commits into
apache:mainfrom
abhijithumbe:kie-7147

Conversation

@abhijithumbe

Copy link
Copy Markdown
Contributor

…n sladuedate format is not correct for the usertask node

Issue: (#7147)

@martinweiler

Copy link
Copy Markdown
Contributor

Thanks for the PR @abhijithumbe ! I have a couple of questions:

  1. I understand we are not trying to address the NPE directly, but rather eliminate the conditions when it could happen, correct?
  2. The fix targets Human Tasks / workItemNodes - what about other node types?
  3. What about process level SLAs?
  4. What happens if user provide an incorrectly formatted date via #{slaVariable}?

…n sladuedate format is not correct for the usertask node
@abhijithumbe

Copy link
Copy Markdown
Contributor Author

Hi @martinweiler,

  1. yes.
  2. Added SLA validation for other nodes.
  3. Added SLA validation for process level SLA
  4. If expression resolution failed I can see a clear SLA parsing error instead of NPE while starting process instance.
|ERROR||io.undertow.request.io|Exception handling request 2216f728-17e9-46e7-94a4-a4667430567a-1 to /TestProcess: org.jboss.resteasy.spi.UnhandledException: java.lang.RuntimeException: Error parsing time string: [ 2026-10-01T18:00:00Z ]
	at org.jboss.resteasy.core.ExceptionHandler.handleApplicationException(ExceptionHandler.java:107)
	at org.jboss.resteasy.core.ExceptionHandler.handleException(ExceptionHandler.java:344)
....
Caused by: java.lang.RuntimeException: Error parsing time string: [ 2026-10-01T18:00:00Z ]
	at org.drools.base.time.TimeUtils.parseTimeString(TimeUtils.java:127)
	at org.jbpm.process.core.timer.DateTimeUtils.parseDuration(DateTimeUtils.java:84)
	at org.jbpm.workflow.instance.impl.WorkflowProcessInstanceImpl.configureSLATimer(WorkflowProcessInstanceImpl.java:594)	

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants