Skip to content

Commit 73bd599

Browse files
committed
fix(ci): prevent Redis CRITICAL crash during build
- Add SKIP_REDIS_CHECK env var guard in rate-limit.ts and distributed-lock.ts - Set SKIP_REDIS_CHECK=1 in Dockerfile builder stage so astro build doesn't crash when Redis is unavailable - Set SKIP_REDIS_CHECK=1 in CI build job for same reason
1 parent f8800fe commit 73bd599

4 files changed

Lines changed: 5 additions & 2 deletions

File tree

.github/workflows/ci.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -305,6 +305,8 @@ jobs:
305305

306306
- name: Build application
307307
run: bun run build
308+
env:
309+
SKIP_REDIS_CHECK: "1"
308310

309311
- name: Upload build artifacts
310312
uses: actions/upload-artifact@v4

Dockerfile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,6 +11,7 @@ RUN bun install --frozen-lockfile
1111
FROM base AS builder
1212
COPY --from=deps /app/node_modules ./node_modules
1313
COPY . .
14+
ENV SKIP_REDIS_CHECK=1
1415
RUN bun run build
1516

1617
# Production image

src/lib/distributed-lock.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import { logger } from "./logger";
99
// Try to import Redis - may not be configured
1010
let redis: any = null;
1111
let redisAvailable = false;
12-
const isProd = process.env.NODE_ENV === "production";
12+
const isProd = process.env.NODE_ENV === "production" && !process.env.SKIP_REDIS_CHECK;
1313

1414
try {
1515
const redisModule = await import("./redis");

src/lib/rate-limit.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import { logger } from "@/lib/logger";
44
// Try to import Redis - may not be configured
55
let redis: any = null;
66
let redisAvailable = false;
7-
const isProd = process.env.NODE_ENV === "production";
7+
const isProd = process.env.NODE_ENV === "production" && !process.env.SKIP_REDIS_CHECK;
88

99
try {
1010
const redisModule = await import("@/lib/redis");

0 commit comments

Comments
 (0)