Skip to content

Fix crash when Lifetime of Execution is 0

Florian Meinicke requested to merge fix-lifetime-expires-immediately into master

Description / What is the current behavior?

When an Observable Command does not specify a Lifetime it should never expire. This case wasn't checked in the ObervableCommandWrapper, however. The Lifetime was just taken as is, i.e. a null-Lifetime resulted in the LifetimeTimer to fire immediately, causing a crash/abort of the ExecutorThread.

Does this MR introduce a breaking changes?

No

Other information

Checklist:

  • The commit messages follow our guidelines
  • Tests have been added so that this bug will be caught in the future
  • CHANGELOG.md has been updated in the Fixed section
Edited by Florian Meinicke

Merge request reports