Merge branch 'main' into redesign-widget

Author: DarianM

frontend/app/utils/validate.server.ts

@@ -2,12 +2,12 @@ import { z } from 'zod'
 import { CornerType, PositionType, SlideAnimationType } from '@shared/types'
 import {
   checkHrefFormat,
-  isWalletAddress,
+  getWalletAddress,
   toWalletAddressUrl,
   WalletAddressFormatError
 } from '@shared/utils'
 
-const rangeError = { message: 'Value has to be between 16 and 24' }
+const rangeError = { message: 'Value has to be between 10 and 30' }
 
 export const walletSchema = z.object({
   walletAddress: z
@@ -19,7 +19,7 @@ export const walletSchema = z.object({
 
       try {
         checkHrefFormat(updatedUrl)
-        await isValidWalletAddress(updatedUrl)
+        await getWalletAddress(updatedUrl)
       } catch (e) {
         ctx.addIssue({
           code: z.ZodIssueCode.custom,
@@ -138,33 +138,3 @@ export const validateForm = async (
 
   return { result, payload }
 }
-
-async function isValidWalletAddress(
-  walletAddressUrl: string
-): Promise<boolean> {
-  const response = await fetch(walletAddressUrl, {
-    headers: {
-      Accept: 'application/json'
-    }
-  })
-
-  if (!response.ok) {
-    if (response.status === 404) {
-      throw new WalletAddressFormatError('This wallet address does not exist.')
-    }
-    throw new WalletAddressFormatError('Failed to fetch wallet address.')
-  }
-
-  const msgInvalidWalletAddress = 'Provided URL is not a valid wallet address.'
-  const json = await response.json().catch((error) => {
-    throw new WalletAddressFormatError(msgInvalidWalletAddress, {
-      cause: error
-    })
-  })
-
-  if (!isWalletAddress(json as Record<string, unknown>)) {
-    throw new WalletAddressFormatError(msgInvalidWalletAddress)
-  }
-
-  return true
-}

shared/utils/index.ts

@@ -1,30 +1,38 @@
 import type { WalletAddress } from '@interledger/open-payments'
 
-export function toWalletAddressUrl(s: string): string {
-  return s.startsWith('$') ? s.replace('$', 'https://') : s
-}
-
 export async function getWalletAddress(
   walletAddressUrl: string
 ): Promise<WalletAddress> {
   const url = toWalletAddressUrl(walletAddressUrl)
 
-  const res = await fetch(url)
-  if (!res.ok) {
-    throw new Error('Unable to fetch wallet details', {
-      cause: new Error(res.statusText || `HTTP ${res.status}`)
+  const response = await fetch(url)
+  if (!response.ok) {
+    if (response.status === 404) {
+      throw new WalletAddressFormatError('This wallet address does not exist')
+    }
+    throw new WalletAddressFormatError('Unable to fetch wallet details', {
+      cause: new WalletAddressFormatError(
+        response.statusText || `HTTP ${response.status}`
+      )
     })
   }
 
+  let json: Record<string, unknown>
   try {
-    const json: Record<string, unknown> = await res.json()
-    if (!isWalletAddress(json)) {
-      throw new Error('Invalid wallet address format')
-    }
-    return json
+    json = await response.json()
   } catch (error) {
-    throw new Error('Failed to parse wallet address content', { cause: error })
+    throw new WalletAddressFormatError(
+      'Provided URL is not a valid wallet address',
+      {
+        cause: error
+      }
+    )
+  }
+  if (!isWalletAddress(json)) {
+    throw new WalletAddressFormatError('Invalid wallet address format')
   }
+
+  return json
 }
 
 export function isWalletAddress(
@@ -44,6 +52,10 @@ export function isWalletAddress(
   )
 }
 
+export function toWalletAddressUrl(s: string): string {
+  return s.startsWith('$') ? s.replace('$', 'https://') : s
+}
+
 export function normalizeWalletAddress(walletAddress: WalletAddress): string {
   const IS_INTERLEDGER_CARDS =
     walletAddress.authServer === 'https://auth.interledger.cards'