Fix: Fail to run route with toD #4972

[claudio4j]

Fix #4972

Release Note

Enhancement of the dependency calculation for dynamic endpoints

[squakez]

LGTM, however I am not understanding how this is solving the issue. I mean, we still will need to include the component dependency manually, right?

[claudio4j]

LGTM, however I am not understanding how this is solving the issue.

This fix only the parsing bug, as the toD notation allows a flexible placeholder, so instead to check only the ending of the string, it if the remaining scheme contains the ending placeholder.

I mean, we still will need to include the component dependency manually, right?

Yes, the camel-k dependency resolver is mostly a regexp parsing the strings, which in the case of the placeholder is not know at this time, only when the code is parsed by camel-core (after camel-k operator build).

[MotaOcimar]

Hi @squakez! Thanks for reviewing!

The full discussion of the issue is at this Zulip topic.

But in short, the following route doesn't work even if we specify the camel-http dependency:

<route id="start-route">
    <from uri="timer:my-timer"/>
    <setBody>
        <constant>Hello!</constant>
    </setBody>
    <toD uri="{{endpoint}}/test"/>
</route>

If we remove /test from the path it works, exemplifying the parsing bug.

I'm hoping it will be approved until the next release 🙏

[claudio4j]

The only CI error check is fixed by #4957