Skip to content

Conversation

@MartinSarkany
Copy link
Contributor

@MartinSarkany MartinSarkany commented Sep 24, 2025

Pull Request Template

Description

Adds synchronous initialization to CosmosDataEncryptionKeyProvider.
Depends on #5418 due to TestEncryptionKeyStoreProvider being introduced there.

Type of change

Please delete options that are not relevant.

  • [] New feature (non-breaking change which adds functionality)

Closing issues

closes #5400

@MartinSarkany MartinSarkany marked this pull request as ready for review September 24, 2025 15:41
@MartinSarkany MartinSarkany changed the title Add synchronous initialization to CosmosDataEncryptionKeyProvider [Client encryption] Add synchronous initialization to CosmosDataEncryptionKeyProvider Sep 24, 2025
Copy link
Contributor

@juraj-blazek juraj-blazek left a comment

Choose a reason for hiding this comment

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

We need to regenerate public API contracts, too

@MartinSarkany
Copy link
Contributor Author

We need to regenerate public API contracts, too

I'll update it after #5373 that fixes the contracts is merged. It is broken at the moment.

@adamnova
Copy link
Contributor

We need to regenerate public API contracts, too

I'll update it after #5373 that fixes the contracts is merged. It is broken at the moment.

But it should work for .net 6 at least.

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.

[Client encryption]: Initialize CosmosDataEncryptionKeyProvider without making a backend call

3 participants