Possible to save bad flow which runs idefinitely

[paulgrainger85]

Describe the issue

Consider the parent and subflow below. Notice that the type is incorrect, it should be Subflow, not subflow. Kestra UI allows us to save this flow.

id: parent
namespace: abc

tasks:
  - id: hello
    type: io.kestra.plugin.core.flow.subflow
    flowId: child
    namespace: abc
-------
id: child
namespace: abc

tasks:
  - id: hello
    type: io.kestra.plugin.core.log.Log
    message: Hello World! 🚀

The subflow is executed correctly, but the parent flow remains running indefinitely (until manually killed).

What should be the correct behavior here? (stop flow from being saved in the first place, or allow case-insensitive type definition)

Environment

• Kestra Version: 0.21.2

[anna-geller]

I'd say we should stop the flow from being saved in the first place here

thx for the report, makes sense!

[yuri1969]

The root cause seems to be the lowercasing of plugin's canonical name introduced by: 929ee7b