Skip to content

Commit bf281eb

Browse files
author
Felipe Zimmerle
committed
Relaxing the audit log sanity checks to allow empty relevant status
1 parent 09ee471 commit bf281eb

File tree

1 file changed

+6
-0
lines changed

1 file changed

+6
-0
lines changed

src/audit_log/audit_log.cc

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -228,9 +228,15 @@ bool AuditLog::init(std::string *error) {
228228
/* Sanity check */
229229
if (m_status == RelevantOnlyAuditLogStatus) {
230230
if (m_relevant.empty()) {
231+
/*
231232
error->assign("m_relevant cannot be null while status is set to " \
232233
"RelevantOnly");
233234
return false;
235+
*/
236+
// FIXME: this should be a warning. There is not point to
237+
// have the logs on relevant only if nothing is relevant.
238+
//
239+
// Not returning an error to keep the compatibility with v2.
234240
}
235241
}
236242

0 commit comments

Comments
 (0)