Cleanup: remove deprecated BlackListManager, unify HttpClient via IHttpClientFactory, deduplicate IPC encryption#420
Merged
Conversation
… instance, deduplicate IPC encryption - Delete deprecated BlackListManager singleton, extract IBlackListMatcher to own file - Add HttpClientProvider with static shared HttpClient (no DI dependency) - Replace all new HttpClient() with HttpClientProvider.Shared - Extract IpcEncryption shared AES utility used by Environments and IProcessInfoProvider - Fix Muti -> Multi file name typo (4 event args) - Delete stale net9.0 bin/obj and ConfiginfoBuilder-Example.cs Closes #419
JusterZhu
force-pushed
the
cleanup/blacklist-httpclient-ipc
branch
from
f948ea9 to
f6f2e8d
Compare
JusterZhu
added a commit
that referenced
this pull request
May 25, 2026
Restore all modified files from 7c3d702 and reapply: - BlackListManager removal (StorageManager, BlackListDefaults) - HttpClientProvider.Shared in strategies - IpcEncryption delegates in Environments, IProcessInfoProvider - IBlackListMatcher extracted to own file - Muti->Multi file rename - Delete ConfiginfoBuilder-Example.cs Closes #419
This was referenced May 25, 2026
JusterZhu
added a commit
that referenced
this pull request
May 25, 2026
…ListManager references - Restore all files from before #420 merge and re-apply changes with proper encoding - Replace BlackListManager.Instance references in Differential project and tests - Fix ParameterMatrixAndEventTests to work with DefaultBlackListMatcher API - Use BlackListDefaults instead of removed BlackListManager singleton All changes from #419 + CI build fixes. Build and tests pass.
JusterZhu
added a commit
that referenced
this pull request
May 25, 2026
Fix encoding corruption from #420 and cross-project BlackListManager references
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Closes #419
Changes
1. Remove deprecated BlackListManager
BlackListManagerobsolete singletonIBlackListMatcherinterface into own file?? BlackListManager.Instancefallback in StorageManager2. Unify HttpClient with IHttpClientFactory
HttpClientFactoryProvider- lazy singleton wrapping ServiceCollection to IHttpClientFactorynew HttpClient()in strategies with factory CreateClient3. Deduplicate IPC encryption
IpcEncryption- shared AES encrypt/decrypt utilityBuild: passes (0 errors)