Skip to content
This repository was archived by the owner on Nov 26, 2022. It is now read-only.

Conversation

@theseion
Copy link
Contributor

@theseion theseion commented Sep 17, 2022

  • The perflogjson format did not exist and the PERFLOG environment variable had no effect because write_perflog was never set.
  • Add comments about predefined log formats

Fixes #128

@theseion theseion requested a review from fzipi September 17, 2022 06:14
@theseion theseion added the bug label Sep 17, 2022
@fzipi
Copy link
Member

fzipi commented Sep 17, 2022

Builds won't wont until we merge #140 I suspect. Or the fix, using LD_LIBRARY_PATH= before the apt-get. But that comes in that PR. 🤷

@theseion
Copy link
Contributor Author

I suspected that when I saw the message about LD_LIBRARY_PATH in the output :)

@fzipi fzipi changed the base branch from master to develop September 17, 2022 16:05
@fzipi fzipi changed the title bugfix: Clean up log formats bugfix: clean up log formats Sep 22, 2022
- The `perflogjson` format did not exist and the `PERFLOG` environment
variable had no effect because `write_perflog` was never set.
- Add comments about predefined log formats
Copy link
Member

@fzipi fzipi left a comment

Choose a reason for hiding this comment

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

LGTM

@fzipi fzipi merged commit 106b2a2 into coreruleset:develop Sep 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add log format definition for perflogjson

2 participants