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

Mongo healthcheck #40

Merged
merged 3 commits into from
Jan 4, 2024
Merged

Mongo healthcheck #40

merged 3 commits into from
Jan 4, 2024

Conversation

xBlaz3kx
Copy link
Contributor

@xBlaz3kx xBlaz3kx commented Jan 4, 2024

Added MongoDB healthcheck.

The tests aren't fully working, as I haven't really figured out why the mock mongo client isn't being created when running tests, but this healthcheck works on a production appplication.

checks/mongo-check_test.go Outdated Show resolved Hide resolved
@tavsec
Copy link
Owner

tavsec commented Jan 4, 2024

@xBlaz3kx thanks for PR! Except for the typo in the test's error message, everything else is good. Could you also add an example in the README.md file for the Mongo healthckes?

@tavsec tavsec merged commit 290653b into tavsec:main Jan 4, 2024
4 checks passed
@xBlaz3kx xBlaz3kx deleted the feature/mongo-check branch January 4, 2024 19:42
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.

2 participants