Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Webhook endpoint may not be fully initialized when the route policy is applied #236

Closed
lburgazzoli opened this issue Feb 4, 2020 · 0 comments · Fixed by #251
Closed

Webhook endpoint may not be fully initialized when the route policy is applied #236

lburgazzoli opened this issue Feb 4, 2020 · 0 comments · Fixed by #251
Assignees
@lburgazzoli
Milestone
1.2.0

Comments

@lburgazzoli
Copy link
Contributor

The WebHookRoutePolicy is triggered when a route is initialized but at this point the underlying web-hook endpojnt may not be fully initialized as the initialization happens on WebhookEndpoint::doStart which could lead to a NPE here and here
@lburgazzoli lburgazzoli added this to the 1.1.0 milestone Feb 4, 2020
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Feb 4, 2020
@lburgazzoli
Disable tests because of apache#236
c3217ad
@lburgazzoli lburgazzoli mentioned this issue Feb 25, 2020
Closed
@lburgazzoli lburgazzoli modified the milestones: 1.1.0, 1.2.0 Mar 2, 2020
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Mar 3, 2020
@lburgazzoli
Disable tests because of apache#236
1ef5b06
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Mar 3, 2020
@lburgazzoli
Webhook endpoint may not be fully initialized when the route policy i…
baa1d7b 
…s applied apache#236
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Mar 4, 2020
@lburgazzoli
Webhook endpoint may not be fully initialized when the route policy i…
89ea20c 
…s applied apache#236
lburgazzoli added a commit to lburgazzoli/apache-camel-k-runtime that referenced this issue Mar 6, 2020
@lburgazzoli
Webhook endpoint may not be fully initialized when the route policy i…
cb41f8d 
…s applied apache#236
@lburgazzoli lburgazzoli mentioned this issue Mar 6, 2020
Merged
lburgazzoli added a commit that referenced this issue Mar 6, 2020
@lburgazzoli
Disable tests because of #236
61e1a83
lburgazzoli added a commit that referenced this issue Mar 6, 2020
@lburgazzoli
Webhook endpoint may not be fully initialized when the route policy i…
8386009 
…s applied #236
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lburgazzoli lburgazzoli

Labels
None yet
Projects
None yet
Milestone
1.2.0
Development

Successfully merging a pull request may close this issue.

Update to camel 3.1, camel-quarkus 1.0.0-M4 lburgazzoli/apache-camel-k-runtime
2 participants
@lburgazzoli @nicolaferraro