mirror of
https://github.com/supabase/supabase.git
synced 2026-06-11 06:19:22 +08:00
## Problem - API may return a non-array shape that can crash `getKeys` because of an hard coded cast - getting API keys is cumbersome as consumers have to call two functions ## Solution - consolidate `useAPIKeysQuery` + `getKeys` into a single `useAPIKeys` hook - guard `getKeys` so that it doesn't crash if passed a non array value - update usages <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **Refactor** * Unified how project API keys are retrieved across the studio, resulting in more consistent loading/error handling and slight responsiveness improvements when showing keys and related command snippets. UI and permissions behavior remain unchanged for end users. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
123 lines
3.2 KiB
TypeScript
123 lines
3.2 KiB
TypeScript
import { useQuery } from '@tanstack/react-query'
|
|
import { useCallback } from 'react'
|
|
|
|
import { apiKeysKeys } from './keys'
|
|
import { get, handleError } from '@/data/fetchers'
|
|
import type { ResponseError, UseCustomQueryOptions } from '@/types'
|
|
|
|
type LegacyKeys = {
|
|
api_key: string
|
|
description?: string | null
|
|
hash?: string | null
|
|
id?: string | null
|
|
inserted_at?: string | null
|
|
name: string
|
|
prefix?: string | null
|
|
secret_jwt_template?: { role: string } | null
|
|
type: 'legacy' | null
|
|
updated_at?: string | null
|
|
}
|
|
|
|
type SecretKeys = {
|
|
api_key: string
|
|
description?: string
|
|
hash: string
|
|
id: string
|
|
inserted_at: string
|
|
name: string
|
|
prefix: string
|
|
secret_jwt_template: { role: string }
|
|
type: 'secret'
|
|
updated_at?: string
|
|
}
|
|
|
|
type PublishableKeys = {
|
|
api_key: string
|
|
description?: string
|
|
hash?: string
|
|
id: string
|
|
inserted_at: string
|
|
name: string
|
|
prefix?: string
|
|
secret_jwt_template?: { role: string } | null
|
|
type: 'publishable'
|
|
updated_at?: string
|
|
}
|
|
|
|
interface APIKeysVariables {
|
|
projectRef?: string
|
|
reveal?: boolean
|
|
}
|
|
|
|
export type APIKey = LegacyKeys | SecretKeys | PublishableKeys
|
|
|
|
async function getAPIKeys({ projectRef, reveal }: APIKeysVariables, signal?: AbortSignal) {
|
|
if (!projectRef) throw new Error('projectRef is required')
|
|
|
|
const { data, error } = await get(`/v1/projects/{ref}/api-keys`, {
|
|
params: { path: { ref: projectRef }, query: { reveal } },
|
|
signal,
|
|
})
|
|
|
|
if (error) handleError(error)
|
|
|
|
// [Jonny]: Overriding the types here since some stuff is not actually nullable or optional
|
|
return data as unknown as APIKey[]
|
|
}
|
|
|
|
export type APIKeysData = Awaited<ReturnType<typeof getAPIKeys>>
|
|
|
|
export const useAPIKeysQuery = <TData = APIKeysData>(
|
|
{ projectRef, reveal = false }: APIKeysVariables,
|
|
{ enabled = true, ...options }: UseCustomQueryOptions<APIKeysData, ResponseError, TData> = {}
|
|
) => {
|
|
return useQuery<APIKeysData, ResponseError, TData>({
|
|
queryKey: apiKeysKeys.list(projectRef, reveal),
|
|
queryFn: ({ signal }) => getAPIKeys({ projectRef, reveal }, signal),
|
|
enabled: enabled && typeof projectRef !== 'undefined',
|
|
...options,
|
|
})
|
|
}
|
|
|
|
const EmptyKey = {
|
|
anonKey: undefined,
|
|
serviceKey: undefined,
|
|
publishableKey: undefined,
|
|
secretKey: undefined,
|
|
allSecretKeys: [],
|
|
}
|
|
|
|
export const getKeys = (apiKeys: APIKey[] = []) => {
|
|
if (!Array.isArray(apiKeys)) {
|
|
return EmptyKey
|
|
}
|
|
const anonKey = apiKeys.find((x) => x.name === 'anon')
|
|
const serviceKey = apiKeys.find((x) => x.name === 'service_role')
|
|
|
|
// [Joshen] For now I just want 1 of each, I don't need all
|
|
const publishableKey = apiKeys.find((x) => x.type === 'publishable')
|
|
const secretKey = apiKeys.find((x) => x.type === 'secret')
|
|
|
|
const allSecretKeys = apiKeys.filter((x) => x.type === 'secret')
|
|
|
|
return { anonKey, serviceKey, publishableKey, secretKey, allSecretKeys }
|
|
}
|
|
|
|
const PlaceholderData: APIKeysData = []
|
|
|
|
export const useAPIKeys = (
|
|
variables: APIKeysVariables,
|
|
options: UseCustomQueryOptions<APIKeysData, ResponseError, APIKeysData> = {}
|
|
) => {
|
|
const select = useCallback(
|
|
(data: APIKeysData) => getKeys(options.enabled ? data : []),
|
|
[options.enabled]
|
|
)
|
|
|
|
return useAPIKeysQuery(variables, {
|
|
...options,
|
|
placeholderData: PlaceholderData,
|
|
select,
|
|
})
|
|
}
|