[JAVA-4739] Condition to check for crypt shared library is incorrect Created: 27/Sep/22 Updated: 28/Oct/23 Resolved: 29/Sep/22 |
|
| Status: | Closed |
| Project: | Java Driver |
| Component/s: | Client Side Encryption |
| Affects Version/s: | None |
| Fix Version/s: | 4.7.2 |
| Type: | Bug | Priority: | Major - P3 |
| Reporter: | Jeffrey Yemin | Assignee: | Jeffrey Yemin |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Description |
|
See this and [this|https://github.com/mongodb/mongo-java-driver/blob/b9251622747231e05d6b1736a06fe45e03f642e5/driver-reactive-streams/src/main/com/mongodb/reactivestreams/client/internal/crypt/CommandMarker.java#L70. It should be:
Consider also how to add at least unit test coverage since there are currently no tests for this. |
| Comments |
| Comment by Githook User [ 29/Sep/22 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@mongodb.com', 'username': 'jyemin'}Message: Fix check for whether crypt shared lib is available Also add unit tests for isMongocryptdSpawningDisabled helper method
|
| Comment by Githook User [ 29/Sep/22 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@mongodb.com', 'username': 'jyemin'}Message: Extract method for duplicated logic checking for whether mongocryptd is disabled
|
| Comment by Githook User [ 29/Sep/22 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@mongodb.com', 'username': 'jyemin'}Message: Fix check for whether crypt shared lib is available Also add unit tests for isMongocryptdSpawningDisabled helper method
|
| Comment by Githook User [ 29/Sep/22 ] |
|
Author: {'name': 'Jeff Yemin', 'email': 'jeff.yemin@mongodb.com', 'username': 'jyemin'}Message: Extract method for duplicated logic checking for whether mongocryptd is disabled
|