Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

Commit

Permalink
Only audit log non-empty appends
Browse files Browse the repository at this point in the history
The smoke tests generate lots of log lines
with empty appends. To prevent this, don't
log audits of appending an empty dataset.
  • Loading branch information
mattrco committed Apr 23, 2015
1 parent 4150164 commit ef044bf
Showing 1 changed file with 9 additions and 8 deletions.
17 changes: 9 additions & 8 deletions backdrop/write/api.py
Original file line number Diff line number Diff line change
Expand Up @@ -348,14 +348,15 @@ def trigger_transforms(data_set_config, data=[], earliest=None, latest=None):


def audit_append(data_set_name, data):
start_at, end_at = parse_bounding_dates(data)
extra = {
'data_set': data_set_name,
'start_at': start_at,
'end_at': end_at,
'datapoints': len(data),
}
app.audit_logger.info("Data append action", extra=extra)
if data:
start_at, end_at = parse_bounding_dates(data)
extra = {
'data_set': data_set_name,
'start_at': start_at,
'end_at': end_at,
'datapoints': len(data),
}
app.audit_logger.info("Data append action", extra=extra)


def audit_delete(data_set_name):
Expand Down

0 comments on commit ef044bf

Please sign in to comment.