Allow users to specify whether they want the violations attachment to be json or csv format.

[blueandgold]

By default, CSV is preferred. But we should allow users to select json, if they prefer or there is a need to output hierarchical data structure. This configurable option should be part of the notification configuration schema.

[arbeit]

How about adding an optional data_format field to the configuration stanza of notifiers? Example

    resources:
        - resource: policy_violations
          should_notify: true
          notifiers:
            # Upload violations to GCS.
            - name: gcs_violations
              configuration:
                # gcs_path should begin with "gs://"
                gcs_path: 'gs://fsv/scanner_violations'
                data_format: 'json'

If that optional data_format field is omitted, CSV will be assumed (i.e. it is the default data format).

[blueandgold]

👍 The optional data_format looks good to me. It allows the user to choose csv for email, and then json for GCS.