[SERVER-39492] Allow array of BSON types in encrypt.bsonType specification Created: 11/Feb/19  Updated: 29/Oct/23  Resolved: 13/Apr/19

Status: Closed
Project: Core Server
Component/s: None
Affects Version/s: None
Fix Version/s: 4.1.11

Type: Improvement Priority: Major - P3
Reporter: Ted Tuckman Assignee: Pawel Terlecki
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
Backwards Compatibility: Fully Compatible
Sprint: Query 2019-04-08, Query 2019-04-22
Participants:
Linked BF Score: 64

 Comments   
Comment by Githook User [ 15/Apr/19 ]

Author:

{'name': 'Pawel Terlecki', 'username': 'pterlecki', 'email': 'pawel.terlecki@mongodb.com'}

Message: SERVER-39492 Fix encrypt_schema_types_test
Branch: master
https://github.com/mongodb/mongo/commit/03171bed9efdf0d25054fcddd936ad04db53f4aa

Comment by Githook User [ 13/Apr/19 ]

Author:

{'email': 'pawel.terlecki@mongodb.com', 'name': 'Pawel Terlecki', 'username': 'pterlecki'}

Message: SERVER-39492 Allow array of BSON types in encrypt.bsonType specification

Changed the IDL definition of EncryptionInfo.bsonType to use a custom type based on
MatcherTypeSet. Since it is a pretty generic type we could move it to
basic_types.idl.
Changed InternalSchemaBinDataEncryptedTypeExpression to use
MatcherTypeSet as well, rather than a single type. Parsing logic of
MatcherTypeSet allowed for a concise code.
Branch: master
https://github.com/mongodb/mongo/commit/008e0e0abcdcdae60a3fee23fb0a17a7c3b0be0d

Comment by David Storch [ 12/Mar/19 ]

pawel.terlecki did you mean to close SERVER-39171 instead of this ticket? I'm re-opening this ticket, since I don't see any changes pushed to the master branch related to this ticket.

Comment by David Storch [ 15/Feb/19 ]

I agree. Let's put this in the backlog, attached to the "query work for mongocryptd" project, and schedule it much later on in the execution of the project.

Comment by Ted Tuckman [ 14/Feb/19 ]

I think that makes sense.

Comment by Nicholas Zolnierz [ 14/Feb/19 ]

This won't show up in Needs Triage because its in an epic, right? I'm voting to keep this consistent with the existing bsonType and allow an array of types. david.storch, ted.tuckman what do you think?

Generated at Thu Feb 08 04:52:13 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.