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

[Feature] Upgrade Rudolph backend to support 2022-24 schema/sync changes #41

Merged
merged 12 commits into from
Mar 26, 2024
Merged

Conversation

radsec
Copy link
Contributor

@radsec radsec commented Mar 26, 2024

to: @MarkVLK @Tejunalwad
cc: @airbnb/rudolph-maintainers

Background

Its about time the sync server gets some TLC and upgrades...this PR is one of several in the coming days to perform some general housekeeping, upgrades, and cleanup of the overall codebase.

This PR is geared mainly at updating the backend and related TF infrastructure to support the latest sync-server schema changes since 2022 up to the latest 2024.2 sync schema.

The changes will be non-breaking and ensure backwards compatibility with legacy Santa agents.

Changes

  • Update backend code to support 2022-2024.2 sync server schema changes
  • Updating related TF infrastructure to support sync server schema changes
  • General housekeeping in updating the Golang version and its deps
    • Removal of legacy dependencies for better code security
    • Added additional unit testing code coverage

Testing

Virtual VM and tested the following:

  • lockdown and monitor modes
  • Published and tested a variety of sample rules which include blocks, silent_blocks,certificates, binaries, and bundle_rules
  • Tested all sync-type behaviors with Rudolph
Screenshot 2024-03-25 at 15 28 48

@radsec radsec requested a review from MarkVLK March 26, 2024 17:50
@radsec radsec self-assigned this Mar 26, 2024
@radsec radsec added enhancement New feature or request bug Something isn't working labels Mar 26, 2024
@radsec radsec requested a review from Tejunalwad March 26, 2024 18:00
@natesinger natesinger self-requested a review March 26, 2024 19:46
Copy link

@natesinger natesinger left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants