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

GODRIVER-1412 Add client metadata support for wrapping libraries #1892

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

prestonvasquez
Copy link
Collaborator

GODRIVER-1412

Summary

Add options.DriverInfo to allow libraries wrapping the Go Driver to append the client metadata generated by the driver when handshaking the server.

Background & Motivation

From DRIVERS-767:

The data warehouse team is tracking driver usage in Atlas and working on a set of charts for that data. As part of that project we would like to begin tracking projects built at MongoDB or maintained by MongoDB employees that wrap drivers. For example, Mongoid, PyMODM, Doctrine, Mongoose, etc. Note that these are wrapping libraries, not applications, so using the appname field is not appropriate

@prestonvasquez prestonvasquez marked this pull request as draft November 14, 2024 21:07
@mongodb-drivers-pr-bot mongodb-drivers-pr-bot bot added the priority-3-low Low Priority PR for Review label Nov 14, 2024
Copy link
Contributor

API Change Report

./v2/mongo/options

compatible changes

(*ClientOptionsBuilder).SetDriverInfo: added
ClientOptions.DriverInfo: added
DriverInfo: added

./v2/x/mongo/driver/auth

compatible changes

HandshakeOptions.OuterLibraryName: added
HandshakeOptions.OuterLibraryPlatform: added
HandshakeOptions.OuterLibraryVersion: added

./v2/x/mongo/driver/operation

compatible changes

(*Hello).OuterLibraryName: added
(*Hello).OuterLibraryPlatform: added
(*Hello).OuterLibraryVersion: added

./v2/x/mongo/driver/topology

compatible changes

WithOuterLibraryName: added
WithOuterLibraryPlatform: added
WithOuterLibraryVersion: added

@prestonvasquez prestonvasquez marked this pull request as ready for review November 15, 2024 20:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority-3-low Low Priority PR for Review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant