Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Signup queueing #2076

Merged
merged 32 commits into from
Feb 5, 2024
Merged

Signup queueing #2076

merged 32 commits into from
Feb 5, 2024

Conversation

dholms
Copy link
Collaborator

@dholms dholms commented Jan 24, 2024

Adds signup queueing to the entryway.

  • Entryway keep track of the number of activated accounts within a given time period
  • If a new signup exceeds its configured limit then that user's account is created, but it is flagged as "deactivated"
  • That user is given a valid refresh token, but their access token has a com.atproto.deactivated scope that only allows them to hit a select few endpoints (including a new endpoint that allows them to check their location in the queue)
  • Accounts over the activation limit are activated in FIFO order. The app instance that holds the leader lock will poll every 30 seconds for deactivated accounts that are under the activation limit. It will then activate those accounts.
  • After activation, a user can refresh their session to get a valid full-scope access token

Followup that adds notification of account activation: #2115

@dholms dholms changed the base branch from main to multi-pds-auth January 24, 2024 00:53
@dholms dholms changed the base branch from multi-pds-auth to entryway-twilio January 24, 2024 00:54
@dholms dholms mentioned this pull request Jan 24, 2024
Base automatically changed from entryway-twilio to multi-pds-auth January 29, 2024 22:35
@dholms dholms marked this pull request as ready for review January 30, 2024 22:56
@dholms dholms merged commit 51d0231 into multi-pds-auth Feb 5, 2024
11 checks passed
@dholms dholms deleted the signup-queueing-take2 branch February 5, 2024 17:13
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.

2 participants