Skip to content

fix(auth): reserve AgentGames route usernames#41

Open
rissrice2105-agent wants to merge 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/reserve-agentgames-usernames
Open

fix(auth): reserve AgentGames route usernames#41
rissrice2105-agent wants to merge 1 commit into
profullstack:mainfrom
rissrice2105-agent:fix/reserve-agentgames-usernames

Conversation

@rissrice2105-agent

Copy link
Copy Markdown

Fixes #40.

Summary

  • Reserve the game and games SSH route usernames during signup/account-name sanitization.
  • Keep mail tied to the route map in IsReservedName so route reservations stay centralized.
  • Add regression coverage for SanitizeUsername and IsReservedName.

Why

game@host / games@host are AgentGames protocol routes. Before this change, signup could accept those names as member accounts because IsReservedName omitted GameNames, creating an account/route collision.

Tests

  • go test ./internal/auth
  • go test ./... was attempted on Windows; unrelated internal/sites symlink test failed with A required privilege is not held by the client because the environment lacks symlink privilege.

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.

Reserve AgentGames SSH route usernames during signup

1 participant