Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-27 Thread via GitHub
jono-morris commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1782791822 Yes certainly, let me take a look. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-27 Thread via GitHub
davsclaus merged PR #11812: URL: https://github.com/apache/camel/pull/11812 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail:

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-27 Thread via GitHub
davsclaus commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1782720150 We also need some work for camel-spring-boot, to make camel.ssl configuration possible via its auto configuration. Its something like threadpool but not exactly the same, but you

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-27 Thread via GitHub
jono-morris commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1782657382 Changes committed changing `camel.main.ssl` to `camel.ssl`. Thanks -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-26 Thread via GitHub
jono-morris commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1781059982 Yes of course. Let me do that. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-26 Thread via GitHub
davsclaus commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1780978851 Can we change `camel.main.ssl` to `camel.ssl` as this is not tied to main only. -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-26 Thread via GitHub
jono-morris commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1780953617 Added camel.main.ssl.xxx, and updated PrepareCamelMainMojo and BaseMainSupport. Thanks. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-26 Thread via GitHub
jono-morris commented on code in PR #11812: URL: https://github.com/apache/camel/pull/11812#discussion_r1373022595 ## core/camel-main/src/main/java/org/apache/camel/main/DefaultConfigurationProperties.java: ## @@ -159,6 +159,15 @@ public abstract class

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-24 Thread via GitHub
jono-morris commented on PR #11812: URL: https://github.com/apache/camel/pull/11812#issuecomment-1777081026 Thanks for the review. Give me a day to make the changes. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

Re: [PR] Camel-15211: add props to configure SSL context parameters [camel]

2023-10-23 Thread via GitHub
davsclaus commented on code in PR #11812: URL: https://github.com/apache/camel/pull/11812#discussion_r1368586564 ## core/camel-main/src/main/java/org/apache/camel/main/DefaultConfigurationProperties.java: ## @@ -159,6 +159,15 @@ public abstract class