[mds-audit] Extremely small audit fix.

[janedotx]

Just removing a redundant line.

PR Checklist

• simple searchable title - [mds-db] Add PG env var, [config] Fix eslint config
• briefly describe the changes in this PR
• mark as draft if should not be merged
• write tests for all new functionality

Impacts

• Provider
• Agency
• Audit
• Policy
• Compliance
• Daily
• Native
• Policy Author

[mdurling]

I wonder if this was meant to validate the audit trip id?

[avatarneil]

I'm not entirely sue what audit_event_id is used for (haven't dug into the audit code much besides /vehicles), but this line is simply a dupe of the line right below it.

[mdurling]

audit_event_id is just a unique uuid assigned to the audit event. Probably not used for much other than tracking in the db (from back before our tables had id columns). I understand it's a dupe. My point was, that perhaps it was a copy-pasta error and should have been isValidAuditTripId(audit_trip_id) to validate the uuid in the route param, but looking at the code it appears that check already happens in some earlier middleware.

[macsj200]

I too wonder if that was there for a reason, and it was just a typo

[avatarneil]

I'm not entirely sue what audit_event_id is used for (haven't dug into the audit code much besides /vehicles), but this line is simply a dupe of the line right below it.

[marie-x]

LGTM