Skip to content

Version X.Y.Z (Date) #1

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

Conversation

P-venkatsai
Copy link

  1. Introduced --output-format option to control log output format.
  2. Fixed bug: Resolved issue when using -p without -r, preventing errors due to uninitialized variables.
  3. Added -c option for custom filter file, enhancing flexibility in log detection.
  4. Improved missing parameter handling by specifying the name of the missing parameter.
  5. Bug fix: Enhanced error handling for file-related issues, providing clearer messages and exception handling.
  6. Added --file-type option for specifying file types when processing a directory, offering greater user control.
  7. Implemented threading for parallel file processing, significantly improving efficiency for different output files.
  8. Enhanced default_filter.json with important filters (SSRF, XXE, command injection).
  9. Added support for JSON log format by introducing jsonLogParser to convert JSON logs to CLF before processing.

Other enhancements:

  • Improved overall code efficiency and readability.
  • Addressed minor bug fixes and optimizations.

1. Introduced --output-format option to control log output format.
2. Fixed bug: Resolved issue when using -p without -r, preventing errors due to uninitialized variables.
3. Added -c option for custom filter file, enhancing flexibility in log detection.
4. Improved missing parameter handling by specifying the name of the missing parameter.
5. Bug fix: Enhanced error handling for file-related issues, providing clearer messages and exception handling.
6. Added --file-type option for specifying file types when processing a directory, offering greater user control.
7. Implemented threading for parallel file processing, significantly improving efficiency for different output files.
8. Enhanced default_filter.json with important filters (SSRF, XXE, command injection).
9. Added support for JSON log format by introducing jsonLogParser to convert JSON logs to CLF before processing.

Other enhancements:
- Improved overall code efficiency and readability.
- Addressed minor bug fixes and optimizations.

Thank you for your valuable contributions!
@RandomRhythm
Copy link
Collaborator

Thank you for the code contributions! I apologize for not addressing this sooner. I hadn't anticipated a pull request while I was still setting up the repository, especially not such a substantial one. Unfortunately, there are some code conflicts with what I had been developing and your contributions, but I can resolve them. I will review each of your documented changes individually and commit them, ensuring you receive credit.

The JSON component is intriguing. As someone who leans towards test-driven development, I would appreciate a sanitized sample of some logs, if possible. Just an entry or two would be enough, or alternatively, I can create my own based on the code you've provided. I plan to test each change thoroughly to prevent any disruptions or unintended behaviors.

@RandomRhythm
Copy link
Collaborator

The jsonLogParser code has been implemented (#9) along with #2, #3, #4, and #5.

RandomRhythm added a commit that referenced this pull request Feb 2, 2025
…ut format is supported. Will expose as argument once JSON is supported.

Added a missing piece of #5 from 3f8048d

Swap strInputPath for strInputFilePath when processing individual file

Signed-off-by: RandomRhythm <RandomRhythm@RhythmEngineering.com>
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.

2 participants