Skip to content

MONGOCRYPT-812 add crypt_shared version in errors#1130

Merged
kevinAlbs merged 3 commits intomongodb:masterfrom
kevinAlbs:M812
Feb 18, 2026
Merged

MONGOCRYPT-812 add crypt_shared version in errors#1130
kevinAlbs merged 3 commits intomongodb:masterfrom
kevinAlbs:M812

Conversation

@kevinAlbs
Copy link
Contributor

Resolves MONGOCRYPT-812

Include the library version in errors from crypt_shared. Changes an error from:

csfle "analyze_query" failed: BSON field 'insert.bad_field' is an unknown field. [Error 2, code 40415]

To:

[crypt_shared 8.0.0] failed: BSON field 'insert.bad_field' is an unknown field. [Error 2, code 40415]

Intended help users identify a too-old crypt_shared on errors encrypting newer syntax.

"csfle" was the old name of the shared library, which was renamed to "crypt_shared" in MONGOCRYPT-434.

@kevinAlbs kevinAlbs marked this pull request as ready for review February 18, 2026 16:00
@kevinAlbs kevinAlbs requested a review from a team as a code owner February 18, 2026 16:00
Copy link
Contributor

@vector-of-bool vector-of-bool left a comment

Choose a reason for hiding this comment

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

LGTM with one comment

Copy link
Contributor

Choose a reason for hiding this comment

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

Is this macro definition used for anything?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah good spot. That was copy-pasta. Removed.

@kevinAlbs kevinAlbs merged commit e2e5f8b into mongodb:master Feb 18, 2026
36 of 37 checks passed
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.

3 participants

Comments