Skip to content

GODRIVER-3884 Use gpg verification for libmongocrypt windows build variant#2404

Merged
prestonvasquez merged 5 commits into
mongodb:masterfrom
prestonvasquez:ci/godriver-3884-update-libmongocrypt-urls-for-1.18.0+
Jun 2, 2026
Merged

GODRIVER-3884 Use gpg verification for libmongocrypt windows build variant#2404
prestonvasquez merged 5 commits into
mongodb:masterfrom
prestonvasquez:ci/godriver-3884-update-libmongocrypt-urls-for-1.18.0+

Conversation

@prestonvasquez
Copy link
Copy Markdown
Member

@prestonvasquez prestonvasquez commented May 28, 2026

GODRIVER-3884

Verify signature of downloaded libmongocrypt tarballs

@github-actions github-actions Bot added the review-priority-normal Medium Priority PR for Review: within 1 business day label May 28, 2026
@mongodb-drivers-pr-bot
Copy link
Copy Markdown
Contributor

API Change Report

No changes found!

@mongodb-drivers-pr-bot
Copy link
Copy Markdown
Contributor

mongodb-drivers-pr-bot Bot commented May 28, 2026

🧪 Performance Results

Commit SHA: a8b99bb

The following benchmark tests for version 6a1f05454c0334000798bded had statistically significant changes (i.e., |z-score| > 1.96):

Benchmark Measurement % Change Patch Value Stable Region H-Score Z-Score
BenchmarkBSONFullDocumentEncoding ops_per_second_min 36.2321 4140.8725 Avg: 3039.5729
Med: 3050.5713
Stdev: 559.6388
0.7218 1.9679
BenchmarkSingleRunCommand allocated_bytes_per_op -0.2296 12204.0000 Avg: 12232.0833
Med: 12234.0000
Stdev: 7.4010
0.8772 -3.7945

For a comprehensive view of all microbenchmark results for this PR's commit, please check out the Evergreen perf task for this patch.

Comment thread internal/cmd/install-libmongocrypt/main.go Fixed
…ows tarball

  - Bump LIBMONGOCRYPT_TAG to 1.18.2
  - Add GPG signature verification on the Windows download path
  - Keep compile.sh-based install for Linux/macOS (the published Linux/macOS
    release variants are nocrypto and the Go driver does not provide crypto hooks)
@prestonvasquez prestonvasquez force-pushed the ci/godriver-3884-update-libmongocrypt-urls-for-1.18.0+ branch from 52e0843 to f00658a Compare May 29, 2026 15:48
@prestonvasquez prestonvasquez marked this pull request as ready for review May 29, 2026 17:08
@prestonvasquez prestonvasquez requested a review from a team as a code owner May 29, 2026 17:08
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Refactors the libmongocrypt installation flow, bumping the pinned libmongocrypt version from 1.15.1 to 1.18.2 and adding PGP signature verification of the prebuilt tarball on the Windows path. The non-Windows path still clones the upstream repo and runs compile.sh. The Taskfile entry for install-libmongocrypt is reformatted to multi-line.

Changes:

  • Bump LIBMONGOCRYPT_TAG to 1.18.2.
  • On Windows, download the prebuilt tarball, fetch the MongoDB public key, and verify the detached .asc signature in an isolated GNUPGHOME before extracting.
  • Reformat the install-libmongocrypt task definition in Taskfile.yml.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
Taskfile.yml Reformat install-libmongocrypt task command list to multi-line YAML.
etc/install-libmongocrypt.sh Bump libmongocrypt to 1.18.2; add GPG signature verification of the Windows tarball; reindent.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread etc/install-libmongocrypt.sh
@prestonvasquez prestonvasquez changed the title GODRIVER-3884 Refactor install-libmongocrypt to run as go binary GODRIVER-3884 Upgrade to libmongocrypt 1.18.2 + tarball and gpg verification for windows May 29, 2026
@prestonvasquez prestonvasquez changed the title GODRIVER-3884 Upgrade to libmongocrypt 1.18.2 + tarball and gpg verification for windows GODRIVER-3884 Use gpg verification for libmongocrypt windows build variant Jun 2, 2026
Copy link
Copy Markdown
Contributor

@matthewdale matthewdale left a comment

Choose a reason for hiding this comment

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

Looks good! 👍

@prestonvasquez prestonvasquez merged commit affa5d8 into mongodb:master Jun 2, 2026
34 checks passed
@prestonvasquez prestonvasquez deleted the ci/godriver-3884-update-libmongocrypt-urls-for-1.18.0+ branch June 2, 2026 20:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd review-priority-normal Medium Priority PR for Review: within 1 business day

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants