baf822e084
- Add runtime Docker socket permission detection in entrypoint - Detects socket GID and logs helpful deployment guidance - Provides three resolution options (root user, group-add, or chmod) - Non-intrusive: logs only, doesn't modify permissions - Fix notification page routing mismatch - Move notifications route from /notifications to /settings/notifications - Add notifications tab to Settings page with Bell icon - Align navigation structure with route definitions - Enhance Docker API error handling - Return 503 (not 500) when Docker daemon unavailable - Add DockerUnavailableError type for clear error distinction - Implement SSRF hardening (reject arbitrary host values) - Improve security and testability - Move ProxyHost routes to protected auth group - Refactor Docker handler tests to use mocks - Simplify useDocker hook query enablement logic Docker socket fix addresses deployment-level permission issue without code changes. The 503 error correctly signals service unavailability due to configuration, not application bugs. Closes #XX (if applicable)
24 lines
593 B
TypeScript
24 lines
593 B
TypeScript
import { useQuery } from '@tanstack/react-query'
|
|
import { dockerApi } from '../api/docker'
|
|
|
|
export function useDocker(host?: string | null, serverId?: string | null) {
|
|
const {
|
|
data: containers = [],
|
|
isLoading,
|
|
error,
|
|
refetch,
|
|
} = useQuery({
|
|
queryKey: ['docker-containers', host, serverId],
|
|
queryFn: () => dockerApi.listContainers(host || undefined, serverId || undefined),
|
|
enabled: Boolean(host) || Boolean(serverId),
|
|
retry: 1, // Don't retry too much if docker is not available
|
|
})
|
|
|
|
return {
|
|
containers,
|
|
isLoading,
|
|
error,
|
|
refetch,
|
|
}
|
|
}
|