Dmm/add query to protocol report

[Sgtpluck]

🎫 Ticket

Link to the relevant ticket:
Add facial match issuer query to weekly reports

🛠 Summary of changes

This change

• reorders the protocols_report.rb file because it was hard to find things in it
• adds a feature usage table, where we can track feature usage based on API requests
• adds a query to keep track of who is using facial match in production

Note: The reorganization was very noisy, so I split this PR into two commits. The first commit shows the reorganization, the second commit is the meat of the actual change.

[zachmargolis]

Can we add a screenshot of the report to the PR? (with fake/sample data as needed)? Helps to put the changes in context

[zachmargolis]

1. I'd consider calling it "events" (plural) since it's a list
2. updating spacing around success = 1

Suggested change

	event: quote([SAML_AUTH_EVENT, OIDC_AUTH_EVENT]),
	}
	format(<<~QUERY, params)
	fields
	name AS protocol,
	coalesce(properties.event_properties.service_provider, properties.event_properties.client_id) as issuer
	| filter name IN %{event} AND properties.event_properties.success= 1
	events: quote([SAML_AUTH_EVENT, OIDC_AUTH_EVENT]),
	}
	format(<<~QUERY, params)
	fields
	name AS protocol,
	coalesce(properties.event_properties.service_provider, properties.event_properties.client_id) as issuer
	| filter name IN %{events} AND properties.event_properties.success = 1

[Sgtpluck]

sure, i'll fix as long as i'm in here e5d1685

[zachmargolis]

events (plural)

Suggested change

	event: quote([SAML_AUTH_EVENT]),
	}
	format(<<~QUERY, params)
	fields
	properties.event_properties.service_provider AS issuer,
	properties.event_properties.request_signed = 1 AS signed,
	properties.event_properties.request_signed != 1 AS not_signed,
	isempty(properties.event_properties.matching_cert_serial) AND signed AS invalid_signature
	| filter name IN %{event}
	events: quote([SAML_AUTH_EVENT]),
	}
	format(<<~QUERY, params)
	fields
	properties.event_properties.service_provider AS issuer,
	properties.event_properties.request_signed = 1 AS signed,
	properties.event_properties.request_signed != 1 AS not_signed,
	isempty(properties.event_properties.matching_cert_serial) AND signed AS invalid_signature
	| filter name IN %{events}

[Sgtpluck]

fixed in e5d1685

[vrajmohan]

Looks good. Nice work reorganizing the file and cleaning up some headers.

[lmgeorge]

LGTM! (+1 to all of Zach's comments, but none of them are blocking to me.)