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.

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

Feat(mongodb): mongodb driver upgrade #189

Merged
merged 9 commits into from Oct 4, 2023

Conversation

davide-sera
Copy link
Contributor

Pull Request Type

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

Description

This Pull Request includes the following updates:

  • Upgraded the MongoDB driver of CRUD Service from v4.x to v6.x.
  • Updated mongodb-client-encryption from v2.9.0 to v6.0.0.
  • Updated @fastify/mongodb from v6.2.0 to v8.0.0.

The motivation for these updates is to keep the CRUD Service dependencies up-to-date and to take advantage of new features and improvements in the MongoDB driver and related packages.

This PR also includes necessary changes to the Dockerfile to ensure that the updated mongodb-client-encryption version is used in the Docker image.

PR Checklist

  • The commit message follows our guidelines included in the CONTRIBUTING.md
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

Davide Serafino added 4 commits September 28, 2023 14:18
@davide-sera davide-sera self-assigned this Sep 28, 2023
Copy link
Collaborator

@danibix95 danibix95 left a comment

Choose a reason for hiding this comment

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

Great job! Just some small comments on the latest versions and we are ready to go

lib/CrudService.js Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
tests/createConnection.test.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@danibix95 danibix95 left a comment

Choose a reason for hiding this comment

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

Great!

@danibix95 danibix95 merged commit 8b1c0bc into main Oct 4, 2023
9 checks passed
@danibix95 danibix95 deleted the feat/mongodb-driver-upgrade-v6 branch October 4, 2023 11:23
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.

None yet

2 participants