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.

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

✨ CAMEL-18449 platform-http servlet implementation #621

Merged
merged 1 commit into from Sep 5, 2022

Conversation

Croway
Copy link
Contributor

@Croway Croway commented Sep 2, 2022

The idea behind this PR is to use camel-platform-http-starter and camel-servlet-starter without any additional configuration.
If camel-servlet is configured in the context (camel-servlet-starter is declared as a dependency), then a PlatformHttpEngine implementation is created using the servlet, I'm not 100% sure about the reflection, if you think it can be done in some other way please let me know.

@Croway Croway marked this pull request as draft September 2, 2022 15:14
@davsclaus
Copy link
Contributor

You can have dependency on camel-servlet that is okay, then you do not need to use reflection

@Croway Croway marked this pull request as ready for review September 5, 2022 10:36
@Croway
Copy link
Contributor Author

Croway commented Sep 5, 2022

Thank you @davsclaus, I've updated it and created the following example apache/camel-spring-boot-examples#83

@davsclaus davsclaus merged commit 21b5f0f into apache:main Sep 5, 2022
bsmolicek pushed a commit to jboss-fuse/camel-spring-boot that referenced this pull request Dec 2, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants