Title logging

[clareconstantine]

Description

As a followup to #32269, move the logging for WebPurify/Geocoder errors in our profanity/privacy filtering to Firehose. We've been using slog, which is not the current preferred place for logging.

More info on slog:

• https://github.com/code-dot-org/code-dot-org/blob/staging/dashboard/config/deployment.rb#L3
• https://github.com/code-dot-org/code-dot-org/blob/staging/lib/cdo.rb#L153
• https://github.com/code-dot-org/code-dot-org/blob/staging/lib/cdo/slog.rb#L14

Open Question: Should I move the other "slogging" in this file to Firehose as well? https://github.com/code-dot-org/code-dot-org/blob/staging/dashboard/app/controllers/activities_controller.rb#L128

Links

• jira

Testing story

Updated activities_controller.rb unit tests to reflect the new logging.

Reviewer Checklist:

• Tests provide adequate coverage
• Code is well-commented
• New features are translatable or updates will not break translations
• Relevant documentation has been added or updated
• User impact is well-understood and desirable
• Pull Request is labeled appropriately
• Follow-up work items (including potential tech debt) are tracked and linked

[islemaster]

Further history of slog: It stands for "structured logging," and was added by @geoffrey-elliott back in 2014 (before the history of our current repo). Apparently these do (or did) flow into a service called Logentries for later analysis.

We've still got this five-year-old comment on the activity monitor:

code-dot-org/bin/cron/activity-monitor

Lines 6 to 11 in 1a9da02

	# Currently data is sent to LogEntries for collection and collation via TCP-based log. This
	# script collects a 4 hour period worth of data, analyzes it, and reports.
	#
	# FUTURE THOUGHTS: This is the only reason we use LogEntries. We could probably just use a
	# an RDS database, MongoDB, or even just a regular log file and eliminate the cost of
	# LogEntries.

At this point I'm not sure anyone still looks at Logentries, or even has access to it - I'm asking the team now.

[islemaster]

I feel good about this change, but no pressure to go clean up other instances of slog for now.