Skip to content

Remove deprecated temporary analytics API and table#604

Merged
ety001 merged 1 commit into
masterfrom
remove-temporary-analytics-api
Jun 4, 2026
Merged

Remove deprecated temporary analytics API and table#604
ety001 merged 1 commit into
masterfrom
remove-temporary-analytics-api

Conversation

@ety001

@ety001 ety001 commented Jun 3, 2026

Copy link
Copy Markdown
Member

Summary

  • Remove the legacy GET /api/analytics endpoint, handler, database helpers, and unused frontend client call.
  • Drop the Sequelize analytics model and add migration 20260603120000-drop-analytics to remove the analytics table from faucet MySQL.
  • Remove ANALYTICS_UPDATE_SUPERKEY from .env.example.

Operational metrics (event_id 6–11 and others) are written to the report database by the merged data-analysis service. Scalyr shows zero faucet /api/analytics hits over the past two years.

Test plan

  • pnpm run lint
  • Deploy to dev and run pnpm run db:migrate to apply drop migration
  • Verify signup flow (/check_username, /create_account) still works
  • Remove ANALYTICS_UPDATE_SUPERKEY from EB secrets after deploy (orchestration/deployment/dev/faucet/.ebextensions/021-application-env-secrets.config)

Metrics now live in the report service database; Scalyr shows no faucet
/api/analytics traffic in the past two years.
@ety001 ety001 merged commit 5d066f3 into master Jun 4, 2026
1 check passed
@ety001 ety001 deleted the remove-temporary-analytics-api branch June 4, 2026 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant