Skip to content

Conversation

@g2vinay
Copy link
Member

@g2vinay g2vinay commented Apr 24, 2025

Adds Perf optimizations:

  1. Adds Conscrypt. (Required for TLS layer crypto bottlenecks).
  2. Adds a Caching Trust Manager. (Required to not spend repetitive time on validating cert that have already been validated.).
  3. Adds a Custom SSL Context. (Required to specify high performance ciphers to be used in crypto layer).

<dependencies>
<dependency>
<groupId>org.conscrypt</groupId>
<artifactId>conscrypt-openjdk</artifactId>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd really rather not ship with this as a compile dependency. I'd rather offer guidance on using this to improve performance. This library hasn't shipped in a long time and if any CVEs against it come up we may be stuck.

@github-actions
Copy link
Contributor

Hi @g2vinay. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@github-actions github-actions bot added the no-recent-activity There has been no recent activity on this issue. label Jun 27, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Jul 4, 2025

Hi @g2vinay. Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing /reopen if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.

@github-actions github-actions bot closed this Jul 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clientcore no-recent-activity There has been no recent activity on this issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants