-
Notifications
You must be signed in to change notification settings - Fork 1.8k
feat(NODE-5169): Implement emergency logger #3610
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
Merged
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
a8d42ad
refactor(NODE-4813): WIP
W-A-James 6b053d9
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James e0b1db2
refactor(NODE-4813): start first draft of logger implementation
W-A-James d2ff692
style(NODE-4813): eslint
W-A-James a52b1ae
refactor(NODE-4813): Add changes to MongoLogger and ConnectionClosedE…
W-A-James 16fee3f
test(NODE-4813): WIP - start adding tests
W-A-James b88003e
fix(NODE-4813): use HostAddress
W-A-James 06a1bf0
fix(NODE-4813): Add Loggable interface
W-A-James 3501a35
test(NODE-4813): Add connection events tests
W-A-James bf591b9
fix(NODE-4813): omit nullish fields
W-A-James 36e2e32
test(NODE-4813): update tests
W-A-James 80fbecd
refactor(NODE-4813): Update mongo client to accept @@mdb.internalLogC…
W-A-James da4f89c
test(NODE-4813): Add integration tests
W-A-James 3c76aeb
test(NODE-4813): ensure that mongo logger works with MongoDBLogWritab…
W-A-James a39965f
style(NODE-4813): remove todo comment
W-A-James 8bf2176
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James 97c8eba
refactor(NODE-5169): Change name getter on events to a property
W-A-James bcd88a8
Merge branch 'NODE-4813/Implement_severity_logging_methods' of github…
W-A-James d4b6300
refactor(NODE-5169): refactor to make use of existing constants
W-A-James ac7223c
test(NODE-5169): Updates tests for ConnectionClosedEvent logging
W-A-James f46ab9b
refactor(NODE-5169): Update logging of ConnectionClosedEvent to inclu…
W-A-James c7d8f8d
refactor(NODE-5619): continue refactor to use minimal writable interface
W-A-James 993058f
refactor(NODE-5619): fix janky typecasting
W-A-James 124702e
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James 0d91ccd
style(NODE-5169): eslint
W-A-James 0fe2d2b
test(NODE-5169): Remove unneeded tests
W-A-James 6ff5413
Merge branch 'NODE-4813/Implement_severity_logging_methods' of github…
W-A-James 0a5b9cd
refactor(NODE-5169): Start implementing review changes
W-A-James bdde7cf
fix(NODE-5169): review comments
W-A-James f11f9d2
test(NODE-5169): add feature flag tests
W-A-James e70de2a
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James 3c742d6
refactor(NODE-5169): Change log signature
W-A-James 4db07e6
fix(NODE-5169): fix check
W-A-James 896a010
test(NODE-5169): test wip
W-A-James af2d8ed
refactor(NODE-5169): Refactor of resolveOptions and DEFAULT_LOG_TRANS…
W-A-James 3e262c0
refactor(NODE-5169): Export appropriate constants
W-A-James 18ed1e8
refactor(NODE-5169): Address review comments
W-A-James 666221b
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James b76ba6d
fix(NODE-5169): Remove createdStdioLogger from export
W-A-James 4db4913
refactor(NODE-5169): Addres review comments
W-A-James be04325
test(NODE-5169): remove unneeded test file
W-A-James 2299478
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
nbbeeken 57a643c
fix(NODE-5169): Addres review comments and fix ConnectionCheckOutFail…
W-A-James c456ce9
fix(NODE-5169): update signature
W-A-James f258901
fix(NODE-5169): convert to MongoError
W-A-James 442557b
style(NODE-5169): eslint
W-A-James dd66ea3
refactor(NODE-5169): update function signature
W-A-James 369216d
Merge branch 'main' into NODE-4813/Implement_severity_logging_methods
W-A-James File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.