refactor: use NetworkID in faucet calculations

packages/xrpl/HISTORY.md

@@ -5,9 +5,10 @@ Subscribe to [the **xrpl-announce** mailing list](https://groups.google.com/g/xr
 ## Unreleased
 
 ### Fixed
-* `OracleSet` transaction accepts hexadecimal string values for `AssetPrice` field
+* Fix `OracleSet` transaction to accept hexadecimal string values for `AssetPrice` field
 * `TransactionStream` model includes `hash` field in APIv2
 * `TransactionStream` model includes `close_time_iso` field only for APIv2
+* Better faucet support
 
 ## 4.2.0 (2025-2-13)
 

packages/xrpl/src/Wallet/defaultFaucets.ts

@@ -16,11 +16,16 @@ export enum FaucetNetwork {
   Devnet = 'faucet.devnet.rippletest.net',
 }
 
-export const FaucetNetworkPaths: Record<string, string> = {
+export const faucetNetworkPaths: Record<string, string> = {
   [FaucetNetwork.Testnet]: '/accounts',
   [FaucetNetwork.Devnet]: '/accounts',
 }
 
+export const faucetNetworkIDs: Map<number, FaucetNetwork> = new Map([
+  [1, FaucetNetwork.Testnet],
+  [2, FaucetNetwork.Devnet],
+])
+
 /**
  * Get the faucet host based on the Client connection.
  *
@@ -31,19 +36,19 @@ export const FaucetNetworkPaths: Record<string, string> = {
 export function getFaucetHost(client: Client): FaucetNetwork | undefined {
   const connectionUrl = client.url
 
-  // 'altnet' for Ripple Testnet server and 'testnet' for XRPL Labs Testnet server
-  if (connectionUrl.includes('altnet') || connectionUrl.includes('testnet')) {
-    return FaucetNetwork.Testnet
-  }
-
-  if (connectionUrl.includes('sidechain-net2')) {
+  if (client.networkID == null) {
     throw new XRPLFaucetError(
-      'Cannot fund an account on an issuing chain. Accounts must be created via the bridge.',
+      'Cannot create faucet URL without networkID or the faucet_host information',
     )
   }
 
-  if (connectionUrl.includes('devnet')) {
-    return FaucetNetwork.Devnet
+  if (faucetNetworkIDs.has(client.networkID)) {
+    return faucetNetworkIDs.get(client.networkID)
+  }
+
+  // 'altnet' for Ripple Testnet server and 'testnet' for XRPL Labs Testnet server
+  if (connectionUrl.includes('altnet') || connectionUrl.includes('testnet')) {
+    return FaucetNetwork.Testnet
   }
 
   throw new XRPLFaucetError('Faucet URL is not defined or inferrable.')
@@ -54,11 +59,11 @@ export function getFaucetHost(client: Client): FaucetNetwork | undefined {
  *
  * @param hostname - hostname.
  * @returns A String with the correct path for the input hostname.
- * If hostname undefined or cannot find (key, value) pair in {@link FaucetNetworkPaths}, defaults to '/accounts'
+ * If hostname undefined or cannot find (key, value) pair in {@link faucetNetworkPaths}, defaults to '/accounts'
  */
-export function getDefaultFaucetPath(hostname: string | undefined): string {
+export function getFaucetPath(hostname: string | undefined): string {
   if (hostname === undefined) {
     return '/accounts'
   }
-  return FaucetNetworkPaths[hostname] || '/accounts'
+  return faucetNetworkPaths[hostname] || '/accounts'
 }

packages/xrpl/src/Wallet/fundWallet.ts

@@ -3,11 +3,7 @@ import { isValidClassicAddress } from 'ripple-address-codec'
 import type { Client } from '../client'
 import { XRPLFaucetError } from '../errors'
 
-import {
-  FaucetWallet,
-  getFaucetHost,
-  getDefaultFaucetPath,
-} from './defaultFaucets'
+import { FaucetWallet, getFaucetHost, getFaucetPath } from './defaultFaucets'
 
 import { Wallet } from '.'
 
@@ -148,7 +144,7 @@ export async function requestFunding(
   if (!hostname) {
     throw new XRPLFaucetError('No faucet hostname could be derived')
   }
-  const pathname = options.faucetPath ?? getDefaultFaucetPath(hostname)
+  const pathname = options.faucetPath ?? getFaucetPath(hostname)
   const response = await fetch(`https://${hostname}${pathname}`, {
     method: 'POST',
     headers: {

packages/xrpl/test/wallet/fundWallet.test.ts

@@ -2,9 +2,9 @@ import { assert } from 'chai'
 
 import {
   FaucetNetwork,
-  FaucetNetworkPaths,
+  faucetNetworkPaths,
   getFaucetHost,
-  getDefaultFaucetPath,
+  getFaucetPath,
 } from '../../src/Wallet/defaultFaucets'
 import {
   setupClient,
@@ -45,20 +45,20 @@ describe('Get Faucet host ', function () {
   })
 
   it('returns the correct faucetPath for Devnet host', function () {
-    const expectedFaucetPath = FaucetNetworkPaths[FaucetNetwork.Devnet]
+    const expectedFaucetPath = faucetNetworkPaths[FaucetNetwork.Devnet]
     // @ts-expect-error Intentionally modifying private data for test
     testContext.client.connection.url = FaucetNetwork.Devnet
 
     assert.strictEqual(
-      getDefaultFaucetPath(getFaucetHost(testContext.client)),
+      getFaucetPath(getFaucetHost(testContext.client)),
       expectedFaucetPath,
     )
   })
 
   it('returns the correct faucetPath for undefined host', function () {
     const expectedFaucetPath = '/accounts'
 
-    assert.strictEqual(getDefaultFaucetPath(undefined), expectedFaucetPath)
+    assert.strictEqual(getFaucetPath(undefined), expectedFaucetPath)
   })
 
   it('throws if not connected to a known faucet host', function () {