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

Draft: Add Configuration File Option #11

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

malvidin
Copy link

Added a TOML config file option that will be overridden by any CLI options. Moved to use pathlib's Path().rglob() to walk the source directory for submissions. It also accepts single files as submissions.

Also formatted all Python files with Black.

Add default config option file
Add example TOML config
Use pathlib rglob for submission file list
@malvidin malvidin changed the title Add Configuration File Option Draft: Add Configuration File Option Apr 10, 2023
@cccs-kevin cccs-kevin self-requested a review April 11, 2023 12:34
Move analyzer and submitter to consolidated argparse
Move analyzer and submitter to consolidated argparse
Remove click requirement
Use refreshable Client for submitter threads
Validate path argument to submitter
Use logger as first argument to match other methods
@cccs-kevin
Copy link
Collaborator

@malvidin is this still relevant based on your changes to the AL client? CybercentreCanada/assemblyline_client#33

@malvidin
Copy link
Author

This is still relevant. These changes support using a configuration file to set common configurations for the incident manager.
It relies on changes in CybercentreCanada/assemblyline_client#33 to support importing Assemblyline client configuration parsing.
Because this change relies on a change in the client, a more significant version change may be required. The CLI options should be the same, but it requires a matching client.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants