refactor: support for mockgatehub in local environment by bosbaber · Pull Request #2188 · interledger/testnet

bosbaber · 2026-03-24T11:40:43Z

Introduces various changes to the packages to allow for a local environment where we can easily change between pointing to the real Gatehub Sandbox environment, or to a locally hosted mock instance of MockGatehub.

This is part of the greater effort of implementing real Continuous Integration for this project to allow for long term maintenance.

Copilot

Pull request overview

Refactors local development to support MockGatehub + a new local/ Docker stack, and adjusts wallet frontend/backend configuration to work correctly when running inside containers (middleware/SSR using internal hostnames).

Changes:

Add a new local/ docker-compose stack (Postgres/Redis/Rafiki/Mockgatehub) plus helper scripts and env examples.
Update wallet frontend middleware + HTTP client to call the backend via an internal Docker hostname on the server.
Add backend support for overriding GateHub base URLs via GATEHUB_API_BASE_URL, and add a couple of GateHub service race-condition guards.

Reviewed changes

Copilot reviewed 24 out of 26 changed files in this pull request and generated 12 comments.

Show a summary per file

File	Description
pnpm-workspace.yaml	Adds `e2e` workspace entry.
pnpm-lock.yaml	Adds lock entries for the new `e2e` importer + Playwright deps.
packages/wallet/frontend/src/utils/helpers.ts	Changes amount formatting truncation logic.
packages/wallet/frontend/src/middleware.ts	Replaces `userService` call with direct `fetch` to internal backend URL.
packages/wallet/frontend/src/lib/httpClient.ts	Switches server-side base URL to internal backend hostname.
packages/wallet/frontend/next.config.js	Introduces `BACKEND_INTERNAL_URL` env mapping + env override for features flag.
packages/wallet/frontend/Dockerfile.dev	Adds build args/ENV for Next public env vars.
packages/wallet/backend/src/middleware/withSession.ts	Adjusts cookie domain logic (avoid `Domain=localhost`).
packages/wallet/backend/src/gatehub/service.ts	Adds early-return checks to reduce concurrent customer/account creation.
packages/wallet/backend/src/gatehub/client.ts	Uses `GATEHUB_API_BASE_URL` when provided (e.g., Mockgatehub).
packages/wallet/backend/src/config/env.ts	Adds `GATEHUB_API_BASE_URL` env var.
local/scripts/rafiki-setup.js	New script to configure Rafiki tenant/assets/liquidity.
local/init/dbinit.sql	New local DB init for multiple service DBs/users.
local/docker-compose.yml	New local compose stack including Mockgatehub.
local/.gitignore	Ignores local pg data artifacts.
local/.env.example	New example env for the local compose stack.
docker/temp/private-key.pem	Removes committed key material.
docker/prod/kratos.yml	Removed legacy prod docker config.
docker/prod/docker-compose.yml	Removed legacy prod docker config.
docker/prod/.env.example	Removed legacy prod env example.
docker/identity.schema.json	Removed legacy Kratos identity schema.
docker/entrypoint.sh	Removed legacy Kratos entrypoint.
docker/dev/kratos.yml	Removed legacy dev docker config.
docker/dev/docker-compose.yml	Removed legacy dev docker config.
docker/dev/.env.example	Removed legacy dev env example.
.github/copilot-instructions.md	Adds repo instructions (currently still referencing removed `docker/dev/*`).

Files not reviewed (1)

pnpm-lock.yaml: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

dragosp1011 · 2026-03-25T08:18:02Z

  const scaledValue = Number(`${value}e-${assetScale}`)
  const flooredValue =
-    Math.floor(Math.round(scaledValue * 10 ** displayScale)) /
+    Math.floor(Math.round(scaledValue * 10 ** (displayScale + 4)) / 10 ** 4) /


why + 4 is added here?

I was messing around, forgot to take it out.
I was fighting with some floating point issues causing my tests to fail. I think we can live without it.

dragosp1011 · 2026-03-25T08:22:21Z

+  }
+  if (cookieVal) {
+    try {
+      const meRes = await fetch(`${backendUrl}/me`, {


use existent service method for the request await userService.me( ${cookieName}=${req.cookies.get(cookieName)?.value} )

will now use use userService.me() instead of raw fetch

dragosp1011 · 2026-03-25T08:28:36Z

package.json scripts need to be updated based on the new docker compose file

update compose path to ./local/docker-compose.yml, remove stale prod scripts

dragosp1011 · 2026-03-25T08:29:38Z

+    const key = line.slice(0, idx).trim()
+    const value = line.slice(idx + 1).trim()
+    result[key] = value
+  }


new line before return and after block

add braces to one-line if statements, added blank line before return

dragosp1011 · 2026-03-25T08:30:48Z

+// ---- helpers ---------------------------------------------------------------
+function loadDotEnv(envPath) {
+  const result = {}
+  if (!fs.existsSync(envPath)) return result


use blocks for one line if statements as well.

dragosp1011 · 2026-03-26T12:45:29Z

  const scaledValue = Number(`${value}e-${assetScale}`)
  const flooredValue =
-    Math.floor(Math.round(scaledValue * 10 ** displayScale)) /
+    Math.floor(Math.round(scaledValue * 10 ** displayScale) / 10 ** 4) /


is this change needed?

Good point. Thank you.
formatAmount is now the same as that of main.

dragosp1011 · 2026-03-27T11:42:19Z

    "lint:fix": "eslint --max-warnings=0 --fix .",
-    "compose": "docker compose -f ./docker/dev/docker-compose.yml",
-    "compose:prod": "docker compose -f ./docker/prod/docker-compose.yml",
+    "compose": "docker compose -f ./local/docker-compose.yml",


as FE were added to dev compose files, the dev commands need no be updated

After a long back and forth with myself on this topic I decided to make the FE run on the host and not in docker. The reason is because when running FE in docker on a Mac then the watch mechanism does always work as expected. So in the end I reverted back to have the applications run on the host for the local environment.

dragosp1011 · 2026-03-27T11:44:04Z

seems to not be in sync with current branch

dragosp1011 · 2026-03-27T11:59:50Z

+RATE_LIMIT_LEVEL=LAX
+GATEHUB_ACCOUNT_PRODUCT_CODE=DEFAULT
+GATEHUB_CARD_PRODUCT_CODE=DEFAULT
+GATEHUB_NAME_ON_CARD=TEST USER


GATEHUB_NAME_ON_CARD: [ 'Only alphanumeric characters are allowed' ], change to TESTUSER maybe

dragosp1011 · 2026-03-27T12:05:40Z

+      REDIS_URL: ${REDIS_URL:-redis://redis:6379/0}
+      GATEHUB_API_BASE_URL: ${GATEHUB_API_BASE_URL:-http://mockgatehub:8080}
+      GATEHUB_ENV: ${GATEHUB_ENV:-sandbox}
+      GATEHUB_IFRAME_BASE_URL: ${GATEHUB_IFRAME_BASE_URL:-http://localhost:8080}


the env var is defined but it is not used in the app, resulting in incorrect iframe link on FE http://mockgatehub:8080/?bearer=iframe-token-4dfcdb88-fe38-4906-9b34-fb3a832a3e30

dragosp1011 · 2026-03-27T12:35:35Z

+      - ./init/dbinit.sql:/docker-entrypoint-initdb.d/init.sql
+
+  # MockGatehub - Mock Gatehub API service for local development
+  mockgatehub:


on manual deposits the webhook has "deposit_type":"external" , it should have a different value - this is causing double entries in trx list now

another issue: on trx, the account balance show double of the amount in the trx - but the gain should actually be 0 as the trx is between 2 WA in the same account

looks like the balance is increased even when it should be decreased in a trx

So it turns out that Mockgatehub was doing an additional pending webhook which caused the double transaction issue. I checked and this is out of spec and I'm not sure why that ended up in mockgatehub.

I removed it from mockgatehub in 1.12.4 and should be resolved.

sequenceDiagram actor User participant FE as Wallet Frontend participant WBE as Wallet Backend participant MGH as MockGatehub User->>FE: Clicks Deposit on account page FE->>User: Opens DepositDialog popup User->>FE: Fills amount, clicks Submit FE->>WBE: POST /accounts/:accountId/fund {amount: 11} WBE->>MGH: POST /core/v1/transactions {type:1(DEPOSIT), network:8(MANUAL), receiving_address, vault_uuid, amount} MGH->>MGH: Creates transaction (status: PENDING) MGH-->>WBE: Webhook: core.deposit.completed {deposit_type:"external", status:"pending", tx_uuid} MGH->>WBE: 201 Created {transaction} WBE->>WBE: handleDepositWebhook() → INSERT Transaction row #1 into DB WBE->>FE: 200 OK "Account funded" FE->>User: Shows "Deposit success" toast Note over MGH: 2 second delay MGH-->>WBE: Webhook: core.deposit.completed {deposit_type:"external", status:"completed", tx_uuid} MGH->>MGH: Updates balance in MockGatehub store WBE->>WBE: handleDepositWebhook() → INSERT Transaction row #2 into DB ⚠️ BUG User->>FE: Navigates to Transactions page FE->>User: Shows 2 transaction rows for the same deposit

Loading

I do worry that this points to other problems inside TestNet which might need some attention. I will create separate tasks for the following hardening issues which we can discuss separately

only process webhooks where data.status === 'completed'

Before inserting, check whether a Transaction row with paymentId === data.tx_uuid already exists and skip if so

Tymmmy · 2026-04-01T07:53:01Z

Looks like for Safari the page does not go to KYC, it gets stuck in Login (with chrome it gets to the KYC with problem)

Screen.Recording.2026-04-01.at.10.30.12.mov

Tymmmy · 2026-04-01T07:57:05Z

KYC displays:

* feat: Automatically approve blocked deposits * fix: Automatically approve blocked deposits * fix: enforce subject and access ownership checks for grants (#2199) * fix(wallet): enforce subject and access ownership checks for grants * fix(wallet): PR review improvement * refactor: support for mockgatehub in local environment (#2188) * refactor: support for mockgatehub in local environment * fix: fixed linting issues * fix: clean up * fix: repaired broken test related to emailService * fix: removed the +4 precision guard we don't need * fix: address PR review feedback and fix Docker build * fix: removed changes to formatAmount * refactor(local): breaking up into multiple service files * ci: changed configuration scheme to fail fast if misconfigured * ci: makefile target for rafiki assets * ci: default rafiki seeder will only seed two currencies for now * fix: resolved dependency issues and aligned with workspace * refactor: ci now part of local * ci: removal of makefiles * docs: updated readme * lint: fixed all linting problems * fix: cleanup of debug * test: wallet unittest configuration fixes * style: prettier automatic fixes to resolve linting * fix: better secure domain handling * wip: experimenting * refactor: wallet and boutique now run on host * fix: examples are now useful * ci: added npm run to make testing easy * style: linting issues in rafiki setup file * docs: cleanup instructions * style: fixed linting issues introduced when I added th elogging * fix: added validation rules to boutique config * style: fixed linting * style: prettier * fix: removed hardcoded interledger.cards * fix: update dev start * fix: flaky tests resolution * fix: backends will not fail silently * fix: correcting secrets * fix: resolved issue where hosted balances were incorrectly calculated * ci: will now pull before up * fix: we will not ask for sudo if we don't need it anymore * style: prettier lint fix * test: resolved test issue where faker was creating short strings * docs: github copilot instructions update * docs: tweaks to readme * style: prettier fix * fix: removed dangling e2e workspace * fix: minor edit to prevent 'cannot update RefundProvider' bug * fix: solve rafiki connection issues * fix: rafiki assets creation script should fail fast on error * docs: cleaned up documentation and examples * fix: upgraded local environment to use Traefik v3.6 * style: prettier linting * style: did pnpm format * fix(local): traefik will now auth.interledger.test/graphql --------- Co-authored-by: dragosp1011 <dragosh1011@gmail.com> Co-authored-by: Andrei Mihai <andreimihai@Andreis-MacBook-Pro.local> * feat: check if sendingwallet exists * feat: update payload * fix: Format type * fix: update Date Format * fix: Format * fix: add parallel proccessing when approving Transactions * fix: Format --------- Co-authored-by: Andrei Mihai <andrei@interledger.foundation> Co-authored-by: Adrian Boros <adiboros@hotmail.com> Co-authored-by: Stephan Butler <stephan.butler@deepsouthdevels.net> Co-authored-by: dragosp1011 <dragosh1011@gmail.com> Co-authored-by: Andrei Mihai <andreimihai@Andreis-MacBook-Pro.local>

github-actions · 2026-06-15T07:10:54Z

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

github-actions · 2026-06-15T10:58:16Z

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

bosbaber requested a review from Copilot March 24, 2026 11:40

bosbaber self-assigned this Mar 24, 2026

github-actions Bot added package: wallet/frontend Wallet frontend implementations package: wallet/backend Wallet backend implementations type: ci Changes to CI workflows type: docker Issue or improvements regarding the docker environment type: source Source changes labels Mar 24, 2026

Copilot started reviewing on behalf of bosbaber March 24, 2026 11:41 View session

Copilot AI reviewed Mar 24, 2026

View reviewed changes

bosbaber marked this pull request as ready for review March 24, 2026 14:54

bosbaber requested review from Tymmmy and dragosp1011 March 24, 2026 14:54

github-actions Bot added the type: test Improvements or additions to tests label Mar 24, 2026

dragosp1011 reviewed Mar 25, 2026

View reviewed changes

github-actions Bot added the package: boutique/backend Boutique backend implementations label Mar 26, 2026

bosbaber requested a review from dragosp1011 March 26, 2026 04:25

dragosp1011 reviewed Mar 26, 2026

View reviewed changes

bosbaber requested a review from dragosp1011 March 27, 2026 03:57

dragosp1011 reviewed Mar 27, 2026

View reviewed changes

Comment thread pnpm-lock.yaml

dragosp1011 Mar 27, 2026

Copy link
Copy Markdown

Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

seems to not be in sync with current branch

dragosp1011 reviewed Mar 27, 2026

View reviewed changes

bosbaber marked this pull request as draft April 5, 2026 16:58

dragosp1011 and others added 21 commits May 28, 2026 15:09

fix: update dev start

4492109

fix: flaky tests resolution

de60d3d

fix: backends will not fail silently

06e2938

fix: correcting secrets

6c6544f

fix: resolved issue where hosted balances were incorrectly calculated

6144c70

ci: will now pull before up

644c3b2

fix: we will not ask for sudo if we don't need it anymore

f8c2133

style: prettier lint fix

80b2d79

test: resolved test issue where faker was creating short strings

d8a99a5

docs: github copilot instructions update

9af57b2

docs: tweaks to readme

82f0495

style: prettier fix

9b4b6c2

fix: removed dangling e2e workspace

45c348c

fix: minor edit to prevent 'cannot update RefundProvider' bug

972a278

fix: solve rafiki connection issues

779a2fc

fix: rafiki assets creation script should fail fast on error

eaade28

docs: cleaned up documentation and examples

d53a16f

fix: upgraded local environment to use Traefik v3.6

bbe2b2f

style: prettier linting

703df48

style: did pnpm format

2ad2829

fix(local): traefik will now auth.interledger.test/graphql

a96d903

bosbaber force-pushed the stephan/int1-626 branch from 3fb3d02 to a96d903 Compare May 28, 2026 13:11

bosbaber marked this pull request as ready for review May 28, 2026 13:23

dragosp1011 approved these changes May 28, 2026

View reviewed changes

bosbaber merged commit 57787dc into main May 28, 2026
31 of 38 checks passed

bosbaber deleted the stephan/int1-626 branch May 28, 2026 13:27

github-actions Bot added the released label Jun 15, 2026

Uh oh!

Conversation

bosbaber commented Mar 24, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

bosbaber Mar 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Tymmmy commented Apr 1, 2026

Uh oh!

Tymmmy commented Apr 1, 2026

Uh oh!

Uh oh!

github-actions Bot commented Jun 15, 2026

Uh oh!

github-actions Bot commented Jun 15, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

bosbaber commented Mar 24, 2026 •

edited

Loading

bosbaber Mar 25, 2026 •

edited

Loading