automations are now their own common struct. it did not make sense to tie automations with findings. this is done by adding a struct into automations for automations that require their own configuration
iam_revoke: allowdomains not populated
dataset scanner did not lowercase name
router: the idea was each finding type would attempt to unmarshal in the incoming finding. however due to findings being similar the first common finding would succeed. changed Name() to verify the name is correct. in this future this should changed to be a bit more accurate. its odd to have a Name() call also function as a verifier