-
Notifications
You must be signed in to change notification settings - Fork 185
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Properies evaluation broken after the introduction of PropertyBindingSupport #129
Comments
@ppalaga done, please create a test against core too so we can isolate where the problem is |
There is trailing period missing at Line 377 in daaf4d4
Let me figure out how to test that. |
should be part of the constant
On Mon, 12 Aug 2019 at 17:44, Peter Palaga ***@***.***> wrote:
There is trailing period missing at
https://github.com/apache/camel-quarkus/blob/daaf4d455aace992d5ef5cd727c9297c2686d4dd/extensions/core/runtime/src/main/java/org/apache/camel/quarkus/core/runtime/support/FastCamelContext.java#L377
Let me figure out how to test that.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#129?email_source=notifications&email_token=AAOIJBJIWYF7AOZS65OSV5DQEGANPA5CNFSM4ILB2MLKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4C6KLA#issuecomment-520480044>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAOIJBL4LYPCO6I7Z5U3HP3QEGANPANCNFSM4ILB2MLA>
.
--
--
Luca Burgazzoli
|
you can test it by setting:
and test the value is properly propagated to the timer component (which is available in the core it) |
ppalaga
added a commit
to ppalaga/camel-quarkus
that referenced
this issue
Aug 12, 2019
…PropertyBindingSupport
Thanks for the tip! Did exactly that. |
ppalaga
added a commit
to ppalaga/camel-quarkus
that referenced
this issue
Aug 12, 2019
…PropertyBindingSupport
ppalaga
added a commit
to ppalaga/camel-quarkus
that referenced
this issue
Aug 13, 2019
…PropertyBindingSupport
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The evaluation of properties does not work after a76c752 "chore: replace custom properties binding implementation with PropertyBindingSupport from camel support".
Reproducible through building the Twitter itest and running the jar manually. Clearly the twitter credentials need to be set as documented in the itest README. The scenario succeeds before a76c752 and fails after a76c752.
Please assign to me.
The text was updated successfully, but these errors were encountered: