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

Enable SSL for Azure extension #1269 #1294

Merged
merged 1 commit into from
Jun 3, 2020

Conversation

galderz
Copy link
Contributor

@galderz galderz commented Jun 2, 2020

PR for #1269. Replaces #1272.

@galderz
Copy link
Contributor Author

galderz commented Jun 2, 2020

Shall I send a PR to quarkus in parallel to add Camel Azure to the list here?

@jamesnetherton
Copy link
Contributor

Shall I send a PR to quarkus in parallel to add Camel Azure to the list here?

@ppalaga WDYT? There's actually a few other camel extensions that enable SSL support. Not sure if we should document that over on the Quarkus side or whether it'd be better to do it exclusively in our own documentation (probably the better option IMO).

@ppalaga
Copy link
Contributor

ppalaga commented Jun 3, 2020

Shall I send a PR to quarkus in parallel to add Camel Azure to the list here?

@ppalaga WDYT? There's actually a few other camel extensions that enable SSL support. Not sure if we should document that over on the Quarkus side or whether it'd be better to do it exclusively in our own documentation (probably the better option IMO).

Yeah, I think syncing to Quarkus docs is not sustainable. The question is how detailed we should document it on our side. I have created #1300 Let's discuss there so that we do not block this PR.

@jamesnetherton jamesnetherton merged commit 8c4d478 into apache:master Jun 3, 2020
@galderz
Copy link
Contributor Author

galderz commented Jun 5, 2020

Thanks all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants