mirror of
https://github.com/supabase/supabase.git
synced 2026-06-13 10:09:12 +08:00
## Summary - Converts ~27 `executeSql` call sites in `apps/studio/data/**` to build SQL through `safeSql` / `ident` / `literal` / `keyword` / `joinSqlFragments` instead of raw template-string interpolation. - Tightens the `useDatabaseCronJobCreateMutation` and `useDatabaseEventTriggerCreateMutation` `sql`/`query` parameter types from `string` to `SafeSqlFragment` (callers already produce one). - Updates `getDeleteEnumeratedTypeSQL` in `packages/pg-meta` to return `SafeSqlFragment`. - Fixes a bug noticed while testing where Queues integration does not correctly handle queues with uppercase names. ## Pages to manually test - Integrations > Cron Jobs - Integrations > Queues - Database > Triggers > Event Triggers - Database > Indexes - Reports > Query Performance - Storage <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit ## Release Notes * **Bug Fixes** * Queue lookups now correctly handle case-insensitive queue names. * Queue table references are now properly managed and consistently applied throughout the queue management interface. * Improved queue name display normalization in the user interface. * **Chores** * Enhanced SQL query safety across the database layer through parameterized query construction and safer templating approaches. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
89 lines
2.6 KiB
TypeScript
89 lines
2.6 KiB
TypeScript
import dayjs from 'dayjs'
|
|
import { Check, Loader2, X } from 'lucide-react'
|
|
|
|
import { pgmqQueueTable } from './Queues.utils'
|
|
import { useQueuesMetricsQuery } from '@/data/database-queues/database-queues-metrics-query'
|
|
import { PostgresQueue } from '@/data/database-queues/database-queues-query'
|
|
import { useTablesQuery } from '@/data/tables/tables-query'
|
|
import { useSelectedProjectQuery } from '@/hooks/misc/useSelectedProject'
|
|
import { DATETIME_FORMAT } from '@/lib/constants'
|
|
|
|
export interface QueueWithMetrics extends PostgresQueue {
|
|
id: string // Add unique id for DataGrid
|
|
}
|
|
|
|
interface QueueCellProps {
|
|
queue: QueueWithMetrics
|
|
}
|
|
|
|
export const QueueNameCell = ({ queue }: QueueCellProps) => (
|
|
<div className="flex items-center">
|
|
<span className="truncate" title={queue.queue_name}>
|
|
{queue.queue_name}
|
|
</span>
|
|
</div>
|
|
)
|
|
|
|
export const QueueTypeCell = ({ queue }: QueueCellProps) => {
|
|
const type = queue.is_partitioned ? 'Partitioned' : queue.is_unlogged ? 'Unlogged' : 'Basic'
|
|
return (
|
|
<div className="flex items-center">
|
|
<span className="truncate" title={type.toLowerCase()}>
|
|
{type}
|
|
</span>
|
|
</div>
|
|
)
|
|
}
|
|
|
|
export const QueueRLSCell = ({ queue }: QueueCellProps) => {
|
|
const { data: selectedProject } = useSelectedProjectQuery()
|
|
|
|
const { data: queueTables } = useTablesQuery({
|
|
projectRef: selectedProject?.ref,
|
|
connectionString: selectedProject?.connectionString,
|
|
schema: 'pgmq',
|
|
})
|
|
|
|
const queueTable = queueTables?.find((x) => x.name === pgmqQueueTable(queue.queue_name))
|
|
const isRlsEnabled = !!queueTable?.rls_enabled
|
|
|
|
return (
|
|
<div className="flex items-center">
|
|
{isRlsEnabled ? <Check size={14} className="text-brand" /> : <X size={14} />}
|
|
</div>
|
|
)
|
|
}
|
|
|
|
export const QueueCreatedAtCell = ({ queue }: QueueCellProps) => (
|
|
<div className="flex items-center">
|
|
<span title={queue.created_at}>{dayjs(queue.created_at).format(DATETIME_FORMAT)}</span>
|
|
</div>
|
|
)
|
|
|
|
export const QueueSizeCell = ({ queue }: QueueCellProps) => {
|
|
const { data: selectedProject } = useSelectedProjectQuery()
|
|
|
|
const { data: metrics, isPending: isLoading } = useQueuesMetricsQuery(
|
|
{
|
|
queueName: queue.queue_name,
|
|
projectRef: selectedProject?.ref,
|
|
connectionString: selectedProject?.connectionString,
|
|
},
|
|
{
|
|
staleTime: 30 * 1000, // 30 seconds
|
|
}
|
|
)
|
|
|
|
return (
|
|
<div className="flex items-center">
|
|
{isLoading ? (
|
|
<Loader2 className="animate-spin" size={16} />
|
|
) : (
|
|
<span>
|
|
{metrics?.queue_length} {metrics?.method === 'estimated' ? '(Approximate)' : null}
|
|
</span>
|
|
)}
|
|
</div>
|
|
)
|
|
}
|