Feat changelog #727
Open
Feat changelog #727
Conversation
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
Fix a data race condition in the memoryStore
added new exported unregister session call
importPathRoot function can now support linux and windows file path syntax
EventTimer.Reset implemented using a channel would deadlock if thread A calls Reset more than once while the EventTimer is invoking its action and the action blocks on thread A (i.e. sending to an unbuffered channel that thread A must receive from)
Change EventTimer to reset its Timer without requiring a reset channel
…lidMessages-Configuration Adds support for RejectInvalidMessage configuration.
Message deep copy
Removed debug print
MongoDB Backing Store
Close iterator when getting messages
Support timeout on ssl connection quickfixgo#347
Add SocketTimeout to config/doc.go quickfixgo#349
Replaced direct usage of MessageStore with MessageStoreFactory to create backup stores dynamically. This change ensures that each session has its own instance of the backup store, facilitating better resource management and isolation. - Changed WithBackupStore to accept MessageStoreFactory instead of MessageStore. - Adjusted fileStoreFactory and newFileStore parameters and implementation accordingly.
Renamed the `storeBackup` struct and related methods to `backupStore` for better clarity and consistency. Added logging statements to handle dropped operations gracefully when encountering high traffic. Updated go.mod to include the logging library.
Ensure `newBackupStore` does not initialize when `MessageStore` is nil. Removed redundant `start()` calls and simplified checks for `nil` in backup store methods. This aims to enhance stability by preventing operations on invalid stores and streamlining function logic.
Removed the redundant `operationRefresh` constant and its corresponding case in the `backupStore` to improve code maintainability. The `operationRefresh` was not being utilized, making its presence unnecessary.
This commit renames `persisteMessage` to `persistentMessage` and `backup` to `backupStore` to improve code readability. It also introduces a new factory, `backupStoreFactory`, to streamline the creation of `backupStore` instances and integrates this factory into the `fileStore` structure. These changes enhance code maintainability and ensure better naming consistency across the codebase.
…l use This change modifies the backup_store.go file by making certain constants and struct types public. The constants and struct fields have been renamed with capitalized first letters to ensure they are exported and accessible from other packages. This improves code reusability and maintainability across different parts of the application.
Renamed the PersistentMessage struct to BackupMessage for better clarity and consistency with the naming of the backupStore context. This change affects type declarations, variable names, and function parameters that previously referenced PersistentMessage. No functional behavior has been modified.
Added backup store support to `FileStore` by introducing a `backupStoreFactory` in the test setup. Updated various dependencies in `go.mod` and `go.sum` files to ensure compatibility and enhance functionality.
Added a check to ensure the method does not proceed if the receiver is nil, preventing potential runtime panics.
Replaced the custom logging package "git.5th.im/lb-public/gear/log" with the standard library's "fmt" for a more straightforward logging approach. Updated all related log function calls to use fmt.Printf and fmt.Println accordingly. Additionally, removed the dependency from go.mod and cleaned up go.sum.
pick Preserve original body when resending
…lestore support file store async mode
backup store
cherry pick upstream changement
expose state
- Return error if client key cert file pair incomplete - Add simple test for TLS load with only CA
cherry-pick upstream modification to support unilateral tls
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
25 participants
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.
upgrade go version