Skip to content

Remove public email check endpoint#14499

Draft
raymondjacobson wants to merge 1 commit into
mainfrom
codex/remove-users-check
Draft

Remove public email check endpoint#14499
raymondjacobson wants to merge 1 commit into
mainfrom
codex/remove-users-check

Conversation

@raymondjacobson

Copy link
Copy Markdown
Member

Summary

  • remove the unauthenticated identity-service /users/check route
  • remove web/mobile/common/libs callers and email-in-use query plumbing
  • keep guest account completion through the existing explicit guestEmail localStorage/query-param sign-up path instead of discovering guest status by arbitrary email lookup
  • add a regression assertion that /users/check now 404s

Validation

  • git diff --check
  • node --check packages/identity-service/src/routes/user.js
  • node --check packages/identity-service/test/expressAppTest.js
  • npm run typecheck -w @audius/common -- --pretty false could not run because this worktree has no installed tsc (node_modules absent)
  • attempted npm test -w identity-service -- --grep "does not expose email registration checks", but the wrapper does not support --grep and failed during Docker Postgres startup before tests ran; created test containers were removed

@changeset-bot

changeset-bot Bot commented Jun 23, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: ac5d837

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions

Copy link
Copy Markdown
Contributor

🌐 Web preview ready

Preview URL: https://audius-web-preview-pr-14499.audius.workers.dev

Unique preview for this PR (deployed from this branch).
Workflow run

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant