Log Matched Events

[asatwal]

Trello-975

Create an events filter for logging, so that we log only the events of interest. This will allow targeted logging for diagnostic and debugging purposes.

The events filter is held in a YAML config.

Event filter example:

#
# Allows logging of events that match the type and given filters on the fields
#
# All values are converted to regular expressions for matching
#
# Any filter fields can be defined as long as the field exists in the target event
#
# If there are mutiple filters then at least one must match the event
#
# All filter fields must match the event fields for a filter to match
#
shared:
  event_filters:
    -
      type: (create|update|delete)_entity
      entity_table_name: course_options
      data:
        key: id
        value: 12345

[duncanjbrown]

Great! Comments inline. I think there are a couple more things to think about re nested fields, and I'm not convinced we need to support them at this stage (unless something has changed), so I've proposed keeping this PR minimal and focussed on the use case at hand, ie matching top-level strings in the event payload.

[asatwal]

Great! Comments inline. I think there are a couple more things to think about re nested fields, and I'm not convinced we need to support them at this stage (unless something has changed), so I've proposed keeping this PR minimal and focussed on the use case at hand, ie matching top-level strings in the event payload.

@duncanjbrown - Thanks for the review. Some very useful feedback. I've added corrections and hopefully clarified the nested fields. We can remove the matching of nested fields and match at the top level only, that would simplify.

I've explained the nested fields through examples, so maybe we can add the examples below to the READ.me ? Or we can remove altogether, I'm not too fussed either way. I think it's not too complex, but then again I implemented this. I'll let you decide :-)

[duncanjbrown]

Brill, thanks for changes @asatwal !

[stevenleggdfe]

Looks good to me; I think @duncanjbrown should approve though rather than me.