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.

Sign up for GitHub

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

Jump to bottom

EncryptionPatchItem failing with latest emulator #2823

Closed
j82w opened this issue Oct 25, 2021 · 1 comment · Fixed by #2835
Closed

EncryptionPatchItem failing with latest emulator #2823

j82w opened this issue Oct 25, 2021 · 1 comment · Fixed by #2835
Assignees
@kr-santosh
Labels
bug Something isn't working Client-Side Encryption Track Client Side Encryption Issues. needs-investigation

Comments

@j82w
Copy link
Contributor

j82w commented Oct 25, 2021

Test method Microsoft.Azure.Cosmos.Encryption.EmulatorTests.MdeEncryptionTests.EncryptionPatchItem threw exception:
Microsoft.Azure.Cosmos.CosmosException: Response status code does not indicate success: BadRequest (400); Substatus: 0; ActivityId: e3745c50-7011-4080-b78f-a3d589aeb649; Reason: (Message: {"Errors":["Collection has ClientEncryptionPolicy set, but the document to be written isn't encrypted."]}
ActivityId: e3745c50-7011-4080-b78f-a3d589aeb649, Request URI: /apps/DocDbApp/services/DocDbServer91/partitions/a4cb49a7-38c8-11e6-8106-8cdcd42c33be/replicas/1p/, RequestStats: Microsoft.Azure.Cosmos.Tracing.TraceData.ClientSideRequestStatisticsTraceDatum, SDK: Windows/10.0.22000 cosmos-netstandard-sdk/3.22.0);

https://cosmos-db-sdk-public.visualstudio.com/cosmos-db-sdk-public/_build/results?buildId=37194&view=logs&jobId=fef5d8be-9ecf-5e7a-84d5-f2c0b04a2d13&j=fef5d8be-9ecf-5e7a-84d5-f2c0b04a2d13&t=29c9c104-7def-54b3-f016-d8bb3a0670a7
@j82w j82w added bug Something isn't working Client-Side Encryption Track Client Side Encryption Issues. needs-investigation labels Oct 25, 2021
@j82w j82w mentioned this issue Oct 25, 2021
Merged
@kirankumarkolli
Copy link
Member

Does it mean the client is not passing the required header?

j82w added a commit that referenced this issue Oct 25, 2021
@j82w
It seems the backend is blocking the request because it is not encryp…
318586b 
…ted. The test is expecting it to pass. I pin (#2824)

Two tests are failing with latest emulator. I updated ItemPatchFailureTest to use the new message.

EncryptionPatchItem is being ignored to unblock gates. It's being tracked: #2823
@kr-santosh kr-santosh mentioned this issue Oct 26, 2021
Merged
@kr-santosh kr-santosh self-assigned this Oct 26, 2021
j82w added a commit that referenced this issue Oct 27, 2021
@j82w
It seems the backend is blocking the request because it is not encryp…
8e66c8f 
…ted. The test is expecting it to pass. I pin (#2824)

Two tests are failing with latest emulator. I updated ItemPatchFailureTest to use the new message.

EncryptionPatchItem is being ignored to unblock gates. It's being tracked: #2823
@kr-santosh kr-santosh mentioned this issue Oct 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kr-santosh kr-santosh

Labels
bug Something isn't working Client-Side Encryption Track Client Side Encryption Issues. needs-investigation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Client Encryption: Fixes patch operation to add encryption header Azure/azure-cosmos-dotnet-v3
3 participants
@kirankumarkolli @j82w @kr-santosh