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

MMDBLOOKUP::FIXED:: Don't shut Rsyslog down when mmdb file could not be read/opened #15

Open
wants to merge 25 commits into
base: dev
Choose a base branch
from

Conversation

frikilax
Copy link
Member

Fixed

  • [MMDBLOOKUP] Handle failures to read/access one mmdb file properly and suspend the module for some time instead of crashing Rsyslog with a failed assert

frikilax and others added 25 commits November 22, 2019 16:05
- new mode 'set' to send SET/SETEX commands
- new parameter 'expiration' to send SETEX instead of SET commands (only applicable to 'set' mode)
- fixes to missing frees
various improvements to the way mmdarwin handle fields:

    now can get lookup fields from internal variables ($.)
    user can now define (module-wise) a root container to put all new keys to
    now supports setting values in sub-keys (key="one!two!three" is now treated correctly)

Bug correction:

    when calling mmdarwin multiple times, keys in non-first instances of mmdarwin would not be in the container
…eak and add tests (#7)

- fix bug in ethernet packets parsing
- resolve memory leak during interface init failure (device not freed after post-create error)
- add test 'impcap_bug_ether' to prove ethernet parser fix is working
- fix a use-after-free variable during darwin uuid message extraction
- improve debug/output by logging uuid parse errors
# New Module
- imhiredis : get messages from a Redis server, using R/LPOP or SUBSCRIBE modes
 - Fix Segmentation fault when server is down
 - Add server connexion check while resuming
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.

3 participants