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

Refactor watcher out of singleton pattern #7042

Merged
merged 1 commit into from Apr 16, 2021

Conversation

theopolis
Copy link
Member

My eventual goal is to add an IPC to the Watcher. Before I do this I would like to simplify the code. I noticed that Watcher does not need to be a singleton.

@theopolis theopolis added the refactor Related to osquery code refactoring label Apr 4, 2021
@theopolis theopolis requested review from a team as code owners April 4, 2021 19:57
Copy link
Member

@directionless directionless left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks fine, though I did not compile or play with it. Not sure if you want a c++ person to take a look

@theopolis theopolis merged commit 5c5fe3b into osquery:master Apr 16, 2021
directionless pushed a commit to directionless/osquery that referenced this pull request Apr 18, 2021
aikuchin pushed a commit to aikuchin/osquery that referenced this pull request Jul 11, 2023
…0 to master

* commit '296db8039bcb40e94ef8c5b07497da5132af565c': (35 commits)
  4.8.0 changelog (osquery#7057)
  Handle events optimization edge cases (osquery#7060)
  Add Shortcuts table for Windows (osquery#6994)
  Refactor watcher out of singleton pattern (osquery#7042)
  Fix optimization for multiple queries using the same subscriber (osquery#7055)
  Small events subscriber refactor to increase test coverage (osquery#7050)
  Use epoch and counter for events-based queries (osquery#7051)
  Guard node key to prevent duplicate enrollments (osquery#7052)
  Fix tls_enroll_max_attempts flag name in the documentation (osquery#7049)
  Improve docs on FIM, mention NTFS and Audit, etc. (osquery#7036)
  Change windows calculation for `physical_memory` (osquery#7028)
  Add an option to enable incremental linking on Windows (osquery#7044)
  config: Add docs for the events top-level-key (osquery#7040)
  Add funding link on GitHub page (osquery#7043)
  Update to `chrome_extensions` to not error on uncommon cases (osquery#7032)
  seccomp_events table in osquery (osquery#7006)
  Free using WTSFreeMemoryEx for WTSEnumerateSessionsExW (osquery#7039)
  Correct the example in the windows_events table spec (osquery#7035)
  Remove Buck leftovers for old versions of OpenSSL (osquery#7034)
  Update docs about OpenSSL and TLS behavior (osquery#7033)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
refactor Related to osquery code refactoring
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants