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

Set application name in MongDB driver config #558

Merged
merged 2 commits into from
Aug 27, 2024

Conversation

mmarcon
Copy link

@mmarcon mmarcon commented Aug 26, 2024

With this change, it's possible to see connections coming from Liquibase in the MongoDB log files.

There are no changes in functionality nor in how the MongoDB driver connects to MongoDB. This change is only about providing some extra metadata to make it easy to audit when connections to a cluster are coming from a Liquibase run.

With this change, it's possible to see connections coming from Liquibase
in the MongoDB log files.
@mmarcon
Copy link
Author

mmarcon commented Aug 27, 2024

@vitaliimak thank you for approving my PR. I see that some checks are failing but failures don't seem to be related to my change. Is there something I can do to make them pass?

@vitaliimak
Copy link

Hi, @mmarcon! Thanks for your contribution!
I don't think that you can fix the build issue.
Hey @jandroav @jnewton03 @sayaliM0412 could you please take a look at this PR? Checks are failing even though the maintainer approved the PR.

@jandroav
Copy link

Hi, @mmarcon! Thanks for your contribution! I don't think that you can fix the build issue. Hey @jandroav @jnewton03 @sayaliM0412 could you please take a look at this PR? Checks are failing even though the maintainer approved the PR.

on it

Copy link

sonarcloud bot commented Aug 27, 2024

@jandroav
Copy link

hello @vitaliimak @mmarcon green build

@KushnirykOleh KushnirykOleh merged commit f904171 into liquibase:main Aug 27, 2024
26 of 27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants