Skip to content

feat(auth): add GitHub authentication - #10

Open
axelav wants to merge 25 commits into
mainfrom
feat/github-auth
Open

axelav wants to merge 25 commits into
mainfrom
feat/github-auth

Conversation

@axelav

@axelav axelav commented Aug 12, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace routine invite-code login with GitHub OAuth while preserving existing accounts
  • add closed/public signup, one-time legacy linking, invitations, and four-hour recovery links
  • version sessions for immediate revocation and harden OAuth, token, configuration, and logging behavior
  • upgrade to Rust 1.94 and SQLx 0.9 for SQLite non-transaction migrations
  • document deployment, production readiness, smoke testing, and rollback

Testing

  • cargo fmt --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo test (200 passed)
  • cargo build --release
  • git diff --check

Production prerequisites

  • register the production GitHub OAuth app
  • configure honkytonk-infra secrets and environment
  • repair and verify the ongoing database backup job
  • verify proxy and upstream query-string logging behavior
  • take and restore-test a production SQLite backup
  • run the real-provider browser smoke test in closed mode

See docs/github-auth-production-rollout.md for the gated rollout procedure.

This branch has not been deployed

No deployments
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