{"action":"created","issue":{"url":"https://api.github.com/repos/syndesisio/syndesis/issues/5197","repository_url":"https://api.github.com/repos/syndesisio/syndesis","labels_url":"https://api.github.com/repos/syndesisio/syndesis/issues/5197/labels{/name}","comments_url":"https://api.github.com/repos/syndesisio/syndesis/issues/5197/comments","events_url":"https://api.github.com/repos/syndesisio/syndesis/issues/5197/events","html_url":"https://github.com/syndesisio/syndesis/pull/5197","id":432488963,"node_id":"MDExOlB1bGxSZXF1ZXN0MjY5OTUwMDQw","number":5197,"title":"[Backport 1.6.x] [camel-k] SyndesisHeaderStrategy not loaded into the registry #5189","user":{"login":"backporting[bot]","id":42222807,"node_id":"MDM6Qm90NDIyMjI4MDc=","avatar_url":"https://avatars3.githubusercontent.com/in/15821?v=4","gravatar_id":"","url":"https://api.github.com/users/backporting%5Bbot%5D","html_url":"https://github.com/apps/backporting","followers_url":"https://api.github.com/users/backporting%5Bbot%5D/followers","following_url":"https://api.github.com/users/backporting%5Bbot%5D/following{/other_user}","gists_url":"https://api.github.com/users/backporting%5Bbot%5D/gists{/gist_id}","starred_url":"https://api.github.com/users/backporting%5Bbot%5D/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/backporting%5Bbot%5D/subscriptions","organizations_url":"https://api.github.com/users/backporting%5Bbot%5D/orgs","repos_url":"https://api.github.com/users/backporting%5Bbot%5D/repos","events_url":"https://api.github.com/users/backporting%5Bbot%5D/events{/privacy}","received_events_url":"https://api.github.com/users/backporting%5Bbot%5D/received_events","type":"Bot","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":4,"created_at":"2019-04-12T10:06:48Z","updated_at":"2019-04-12T12:19:14Z","closed_at":null,"author_association":"NONE","pull_request":{"url":"https://api.github.com/repos/syndesisio/syndesis/pulls/5197","html_url":"https://github.com/syndesisio/syndesis/pull/5197","diff_url":"https://github.com/syndesisio/syndesis/pull/5197.diff","patch_url":"https://github.com/syndesisio/syndesis/pull/5197.patch"},"body":"Backport #5190."},"comment":{"url":"https://api.github.com/repos/syndesisio/syndesis/issues/comments/482553476","html_url":"https://github.com/syndesisio/syndesis/pull/5197#issuecomment-482553476","issue_url":"https://api.github.com/repos/syndesisio/syndesis/issues/5197","id":482553476,"node_id":"MDEyOklzc3VlQ29tbWVudDQ4MjU1MzQ3Ng==","user":{"login":"lburgazzoli","id":1868933,"node_id":"MDQ6VXNlcjE4Njg5MzM=","avatar_url":"https://avatars1.githubusercontent.com/u/1868933?v=4","gravatar_id":"","url":"https://api.github.com/users/lburgazzoli","html_url":"https://github.com/lburgazzoli","followers_url":"https://api.github.com/users/lburgazzoli/followers","following_url":"https://api.github.com/users/lburgazzoli/following{/other_user}","gists_url":"https://api.github.com/users/lburgazzoli/gists{/gist_id}","starred_url":"https://api.github.com/users/lburgazzoli/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lburgazzoli/subscriptions","organizations_url":"https://api.github.com/users/lburgazzoli/orgs","repos_url":"https://api.github.com/users/lburgazzoli/repos","events_url":"https://api.github.com/users/lburgazzoli/events{/privacy}","received_events_url":"https://api.github.com/users/lburgazzoli/received_events","type":"User","site_admin":false},"created_at":"2019-04-12T12:19:14Z","updated_at":"2019-04-12T12:19:14Z","author_association":"COLLABORATOR","body":"> > SyndesisHeaderStrategy is not only in that PR, it is already in the [codebase](https://github.com/syndesisio/syndesis/blob/1.6.x/app/connector/support/processor/src/main/java/io/syndesis/connector/support/processor/SyndesisHeaderStrategy.java).\r\n> \r\n> Compare this version with the version in the codebase.\r\n\r\nAh sorry did't check that, will fix\r\n> \r\n> > This pr is needed for camel-k otherwise it won't work as connectors that require SyndesisHeaderStrategy won't find it in the camel registry (remember we do not use spring-boot).\r\n> > The duplication is needed because SyndesisHeaderStrategy is defined in the connector bits but it is required in integration which is a dependency of connectors, we could move SyndesisHeaderStrategy from connectors to runtime but didn't want to do so at this stage to impact camel-k only but if you feel I should move, I will of course.\r\n> \r\n> Is `connector-support-processor` not part of camel-k runtime classpath?\r\n\r\nit is at runtime but not at compile time and it is needed at compile time.\r\n\r\n"}}
{"actor":{"display_login":"lburgazzoli"}}