BED-4128: Fix skip for audit endpoint

[iustinum]

Description

• Fixes an issue where skip parameter is not parsed from the /api/v2/audit endpoint, resulting in log output not starting at the specified offset

Currently, this endpoint only supports using offset to specify skip, as opposed to other endpoints which uses skip. This change enables the endpoint to support both offset and skip and uses whichever one is populated. If they are both populated, the endpoint uses the skip parameter value.

Motivation and Context

This PR addresses: BED-4128

How Has This Been Tested?

Unit tests have been updated

Screenshots (optional):

Types of changes

• Bug fix (non-breaking change which fixes an issue)

Checklist:

• [X] I have met the contributing prerequisites
  • Assigned myself to this PR
  • Added the appropriate labels
  • Associated an issue: https://github.com/SpecterOps/BloodHound/issues/672
  • Read the Contributing guide: https://github.com/SpecterOps/BloodHound/wiki/Contributing
• [X] I have ensured that related documentation is up-to-date
  • Open API docs
  • Code comments (GoDocs / JSDocs)
• [X] I have followed proper test practices
  • Added/updated tests to cover my changes
  • All new and existing tests passed