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

[FEATURE REQ] Allow users to customize SSLContext in azure-core-amqp and eventhubs #39988

Open
ChrisCollinsIBM opened this issue May 1, 2024 · 1 comment
Assignees
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Event Hubs question The issue doesn't require a change to the product in order to be resolved. Most issues start as that

Comments

@ChrisCollinsIBM
Copy link

ChrisCollinsIBM commented May 1, 2024

As requested in #17963 and #17907 we still face this issue today where we can't pass a preferred SSLContext into the EventHubClientBuilder to cascade into where the ConnectionHandler.java creates the SSLContext.

Individual client connections in a large shared JVM shouldn't rely on the SSLContext.getDefault() to cover all use cases and should be configurable on a per-client basis. Ideally this would be exposed in the already very expansive EventProcessorClientBuilder

This is very common in client SDKs to expose (Apache HTTPClient, etc).

Can we get this re-looked at?

I do appreciate that there are some complexities at work here as you may also need to hand this SSLContext into the Blob Container also which has dependencies on the storage SDK and something at the EventProcessorClientBuilder encapsulates both an amqp connection as well as (typically) a storage connection.

@github-actions github-actions bot added customer-reported Issues that are reported by GitHub users external to the Azure organization. needs-triage This is a new issue that needs to be triaged to the appropriate team. question The issue doesn't require a change to the product in order to be resolved. Most issues start as that labels May 1, 2024
@github-actions github-actions bot removed the needs-triage This is a new issue that needs to be triaged to the appropriate team. label May 6, 2024
@joshfree
Copy link
Member

joshfree commented May 6, 2024

@conniey @Azure/azsdk-sb-java can you please follow up and share the reasoning for the previous triage decision?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
customer-reported Issues that are reported by GitHub users external to the Azure organization. Event Hubs question The issue doesn't require a change to the product in order to be resolved. Most issues start as that
Projects
None yet
Development

No branches or pull requests

3 participants