Skip to content

Add numdays parameter, cleanups #19

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

william-stearns
Copy link
Contributor

  • Add --numdays parameter, help text for it, and extend transfer timeout if the user asks for more days than 3. Force to 3 days if user selected 0, 1, or 2 days.
  • Adjust the find command to save a list of files to be sent out to a file (instead of storing the output in a variable.) This handles the case where a huge number of files is found and can no longer fit on a command line.
  • Modify flock lock file to include the destination and number of days requested. This allows multiple zeek_log_transports to run at the same time - handy when the hourly transfer kicks off while a multiday archive transfer is already running.
  • Handle case where source system does not have the "ip" command installed.
  • Cosmetic (comment) changes from "bro" to "zeek".
  • Add flock command to the required list of tools.
    Satisfies Change zeek_log_transport to handle sending a user-requested number of days of logs #18

@william-stearns william-stearns added the enhancement New feature or request label Apr 1, 2025
@william-stearns william-stearns requested a review from joelillo April 1, 2025 19:50
@william-stearns william-stearns self-assigned this Apr 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant