Skip to content

chore: use new dev url for develop env#389

Merged
chaitanyapotti merged 1 commit intomasterfrom
fix/replace-citadel-dev-url
Mar 6, 2026
Merged

chore: use new dev url for develop env#389
chaitanyapotti merged 1 commit intomasterfrom
fix/replace-citadel-dev-url

Conversation

@arch1995
Copy link
Contributor

@arch1995 arch1995 commented Mar 6, 2026

Motivation and Context

  • use citadel dev url for local env.

How has this been tested?

Screenshots (if appropriate):

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

  • My code follows the code style of this project. (run lint)
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.
  • My code requires a db migration.

Note

Low Risk
Low risk constant change, but it alters where development builds send Citadel requests and may affect local dev workflows that expected a localhost service.

Overview
Updates the BUILD_ENV.DEVELOPMENT default Citadel server endpoint by changing CITADEL_SERVER_URL_DEVELOPMENT from a localhost URL to https://api-develop.web3auth.io/citadel-server.

As a result, code paths (e.g. getDefaultCitadelServerUrl in src/core/auth.ts) will route development-environment Citadel traffic to the hosted develop API instead of a locally running service.

Written by Cursor Bugbot for commit b2bbd49. This will update automatically on new commits. Configure here.

@arch1995 arch1995 requested a review from a team as a code owner March 6, 2026 11:15
@chaitanyapotti chaitanyapotti merged commit 1522534 into master Mar 6, 2026
5 checks passed
@chaitanyapotti chaitanyapotti deleted the fix/replace-citadel-dev-url branch March 6, 2026 11:52
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