chore: fix some small issue for deploy (#161)

Sma1lboy · autofix-ci[bot] · web-flow · commit c0cefe42e24e · 2025-03-05T12:33:44.000-06:00
&lt;!-- This is an auto-generated comment: release notes by coderabbit.ai
--&gt;

## Summary by CodeRabbit

- **New Features**
- Updated login flow: Email confirmation is now conditional, allowing
sign-in when email functionality is disabled.
- Introduced dynamic protocol handling for project previews and
container deployments, enabling seamless HTTP/HTTPS transitions.
- Added secure reverse proxy and TLS configurations to enhance
connectivity and security.

- **Chores**
  - Expanded configuration options for database and service settings.
- Improved the build process with enhanced cleanup and modern networking
adjustments.

&lt;!-- end of auto-generated comment: release notes by coderabbit.ai --&gt;

---------

Co-authored-by: autofix-ci[bot] &lt;114827586+autofix-ci[bot]@users.noreply.github.com&gt;
diff --git a/backend/src/auth/auth.service.ts b/backend/src/auth/auth.service.ts
@@ -45,7 +45,7 @@ export class AuthService {
   ) {
     // Read the MAIL_ENABLED environment variable, default to 'true'
     this.isMailEnabled =
-      this.configService.get<string>('MAIL_ENABLED', 'true').toLowerCase() ===
+      this.configService.get<string>('MAIL_ENABLED', 'false').toLowerCase() ===
       'true';
   }
 
@@ -192,7 +192,7 @@ export class AuthService {
       throw new UnauthorizedException('Invalid credentials');
     }
 
-    if (!user.isEmailConfirmed) {
+    if (!user.isEmailConfirmed && this.isMailEnabled) {
       throw new Error('Email not confirmed. Please check your inbox.');
     }
 
diff --git a/backend/src/config/env.validation.ts b/backend/src/config/env.validation.ts
@@ -1,6 +1,27 @@
-import { IsOptional, IsString, IsNumber, IsIn } from 'class-validator';
+import { IsOptional, IsString, IsNumber, IsIn, IsPort } from 'class-validator';
 
 export class EnvironmentVariables {
+  // Database Configuration - all optional
+  @IsOptional()
+  @IsString()
+  DB_HOST?: string;
+
+  @IsOptional()
+  @IsPort()
+  DB_PORT?: string;
+
+  @IsOptional()
+  @IsString()
+  DB_USERNAME?: string;
+
+  @IsOptional()
+  @IsString()
+  DB_PASSWORD?: string;
+
+  @IsOptional()
+  @IsString()
+  DB_DATABASE?: string;
+
   @IsNumber()
   PORT: number;
 
diff --git a/backend/template/react-ts/vite.config.ts b/backend/template/react-ts/vite.config.ts
@@ -30,5 +30,6 @@ export default defineConfig({
     watch: {
       usePolling: true,
     },
+    allowedHosts: true,
   },
 });
diff --git a/codefox-common/package.json b/codefox-common/package.json
@@ -5,20 +5,11 @@
   "main": "dist/cjs/index.js",
   "module": "dist/esm/index.js",
   "types": "dist/types/index.d.ts",
-  "exports": {
-    ".": {
-      "import": "./dist/esm/index.js",
-      "require": "./dist/cjs/index.js",
-      "types": "./dist/types/index.d.ts",
-      "default": "./dist/esm/index.js"
-    },
-    "./dist/*": "./dist/*"
-  },
   "files": [
     "dist"
   ],
   "scripts": {
-    "build": "pnpm run build:cjs && pnpm run build:esm && pnpm run build:types",
+    "build": "rimraf dist && pnpm run build:cjs && pnpm run build:esm && pnpm run build:types",
     "build:frontend": "pnpm run build",
     "build:backend": "pnpm run build",
     "build:cjs": "tsc -p tsconfig.cjs.json",
diff --git a/docker/docker-compose.pord.yml b/docker/docker-compose.pord.yml
@@ -0,0 +1,29 @@
+version: '3'
+
+services:
+  reverse-proxy:
+    image: traefik:v3.3
+    command:
+      - '--api.insecure=true'
+      - '--providers.docker=true'
+      - '--providers.docker.exposedbydefault=false'
+      - '--providers.file.directory=/etc/traefik/config'
+      - '--providers.file.watch=true'
+      - '--entrypoints.web.address=:80'
+      - '--entrypoints.websecure.address=:443'
+    ports:
+      - '80:80'
+      - '443:443'
+      - '9001:8080'
+    volumes:
+      - /var/run/docker.sock:/var/run/docker.sock
+      - /etc/letsencrypt:/etc/letsencrypt
+      - ./traefik-config:/etc/traefik/config
+    networks:
+      - traefik_network
+    extra_hosts:
+      - 'host.docker.internal:host-gateway'
+
+networks:
+  traefik_network:
+    driver: bridge
diff --git a/docker/docker-compose.yml b/docker/docker-compose.yml
diff --git a/docker/traefik-config/services.yml b/docker/traefik-config/services.yml
@@ -0,0 +1,58 @@
+http:
+  routers:
+    frontend:
+      rule: 'Host(`codefox.net`) && !PathPrefix(`/graphql`)'
+      entrypoints:
+        - websecure
+      tls: {}
+      service: frontend
+      priority: 10
+
+    backend:
+      rule: 'Host(`codefox.net`) && PathPrefix(`/graphql`)'
+      entrypoints:
+        - websecure
+      tls: {}
+      service: backend
+      priority: 20
+    redirect-all:
+      rule: 'hostregexp(`{host:.+}`)'
+      entrypoints:
+        - web
+      middlewares:
+        - redirect-to-https
+      service: noop
+
+  services:
+    frontend:
+      loadBalancer:
+        servers:
+          - url: 'http://host.docker.internal:3000'
+
+    backend:
+      loadBalancer:
+        servers:
+          - url: 'http://host.docker.internal:8080'
+
+    noop:
+      loadBalancer:
+        servers:
+          - url: 'http://localhost:9000'
+
+  middlewares:
+    redirect-to-https:
+      redirectScheme:
+        scheme: https
+        permanent: true
+
+    cors:
+      headers:
+        accessControlAllowMethods:
+          - GET
+          - POST
+          - PUT
+          - DELETE
+          - OPTIONS
+        accessControlAllowHeaders:
+          - Content-Type
+          - Authorization
diff --git a/docker/traefik-config/tls.yml b/docker/traefik-config/tls.yml
@@ -0,0 +1,4 @@
+tls:
+  certificates:
+    - certFile: /etc/letsencrypt/live/codefox.net/fullchain.pem
+      keyFile: /etc/letsencrypt/live/codefox.net/privkey.pem
diff --git a/frontend/.env.example b/frontend/.env.example
@@ -1 +1,6 @@
 NEXT_PUBLIC_GRAPHQL_URL=http://localhost:8080/graphql
+
+# TLS OPTION for HTTPS
+TLS=false 
+# TRAEFIK OPTION for HTTPS
+TRAEFIK_DOMAIN=your_domain.com
diff --git a/frontend/src/app/api/runProject/route.ts b/frontend/src/app/api/runProject/route.ts
@@ -7,6 +7,7 @@ import puppetter from 'puppeteer';
 import { useMutation } from '@apollo/client/react/hooks/useMutation';
 import { toast } from 'sonner';
 import { UPDATE_PROJECT_PHOTO_URL } from '@/graphql/request';
+import { TLS } from '@/utils/const';
 
 const runningContainers = new Map<
   string,
@@ -147,12 +148,25 @@ async function buildAndRunDocker(
         console.log(`Running Docker container: ${containerName}`);
 
         // 3. Run the Docker container
-        const runCommand = `docker run -d --name ${containerName} -l "traefik.enable=true" \
-          -l "traefik.http.routers.${subdomain}.rule=Host(\\"${domain}\\")" \
-          -l "traefik.http.services.${subdomain}.loadbalancer.server.port=5173" \
-          --network=codefox_traefik_network -p ${exposedPort}:5173 \
-          -v "${directory}:/app" \
-          ${imageName}`;
+        let runCommand;
+        if (TLS) {
+          runCommand = `docker run -d --name ${containerName} -l "traefik.enable=true" \
+        -l "traefik.http.routers.${subdomain}.rule=Host(\\"${domain}\\")" \
+        -l "traefik.http.routers.${subdomain}.entrypoints=websecure" \
+        -l "traefik.http.routers.${subdomain}.tls=true" \
+        -l "traefik.http.services.${subdomain}.loadbalancer.server.port=5173" \
+        --network=codefox_traefik_network -p ${exposedPort}:5173 \
+        -v "${directory}:/app" \
+        ${imageName}`;
+        } else {
+          runCommand = `docker run -d --name ${containerName} -l "traefik.enable=true" \
+        -l "traefik.http.routers.${subdomain}.rule=Host(\\"${domain}\\")" \
+        -l "traefik.http.routers.${subdomain}.entrypoints=web" \
+        -l "traefik.http.services.${subdomain}.loadbalancer.server.port=5173" \
+        --network=codefox_traefik_network -p ${exposedPort}:5173 \
+        -v "${directory}:/app" \
+        ${imageName}`;
+        }
 
         console.log(`Executing run command: ${runCommand}`);
 
diff --git a/frontend/src/components/chat/code-engine/web-view.tsx b/frontend/src/components/chat/code-engine/web-view.tsx
@@ -12,6 +12,7 @@ import {
   ZoomOut,
 } from 'lucide-react';
 import puppeteer from 'puppeteer';
+import { URL_PROTOCOL_PREFIX } from '@/utils/const';
 
 export default function WebPreview() {
   const { curProject, getWebUrl } = useContext(ProjectContext);
@@ -41,7 +42,7 @@ export default function WebPreview() {
       lastProjectPathRef.current = projectPath;
 
       if (containerRef.current?.projectPath === projectPath) {
-        setBaseUrl(`http://${containerRef.current.domain}`);
+        setBaseUrl(`${URL_PROTOCOL_PREFIX}://${containerRef.current.domain}`);
         return;
       }
 
@@ -52,7 +53,7 @@ export default function WebPreview() {
           domain,
         };
 
-        const baseUrl = `http://${domain}`;
+        const baseUrl = `${URL_PROTOCOL_PREFIX}://${domain}`;
         console.log('baseUrl:', baseUrl);
         setBaseUrl(baseUrl);
         setDisplayPath('/');
diff --git a/frontend/src/components/root/expand-card.tsx b/frontend/src/components/root/expand-card.tsx
@@ -4,6 +4,7 @@ import React, { useContext, useEffect, useRef, useState } from 'react';
 import { AnimatePresence, motion } from 'framer-motion';
 import { X } from 'lucide-react';
 import { ProjectContext } from '../chat/code-engine/project-context';
+import { URL_PROTOCOL_PREFIX } from '@/utils/const';
 
 export function ExpandableCard({ projects }) {
   const [active, setActive] = useState(null);
@@ -37,7 +38,7 @@ export function ExpandableCard({ projects }) {
 
     try {
       const data = await getWebUrl(project.path);
-      const url = `http://${data.domain}`;
+      const url = `${URL_PROTOCOL_PREFIX}://${data.domain}`;
       cachedUrls.current.set(project.id, url);
       setIframeUrl(url);
     } catch (error) {
@@ -120,7 +121,7 @@ export function ExpandableCard({ projects }) {
               const data = await getWebUrl(project.path);
 
               console.log(project.image);
-              const url = `http://${data.domain}`;
+              const url = `${URL_PROTOCOL_PREFIX}://${data.domain}`;
               setIframeUrl(url);
               handleCardClick(project);
               setActive(project);
diff --git a/frontend/src/utils/const.ts b/frontend/src/utils/const.ts
@@ -0,0 +1,12 @@
+/**
+ * @description: API URL
+ * @type {string}
+ * @example 'https://api.example.com'
+ */
+export const URL_PROTOCOL_PREFIX =
+  process.env.TLS == 'false' ? 'http' : 'https';
+
+/**
+ * Validate if the current environment is using TLS
+ */
+export const TLS = process.env.TLS == 'true';
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml

Original file line number	Diff line number	Diff line change
`@@ -45,7 +45,7 @@ export class AuthService {`
`45`	`45`	`) {`
`46`	`46`	`// Read the MAIL_ENABLED environment variable, default to 'true'`
`47`	`47`	`this.isMailEnabled =`
`48`		`- this.configService.get<string>('MAIL_ENABLED', 'true').toLowerCase() ===`
	`48`	`+ this.configService.get<string>('MAIL_ENABLED', 'false').toLowerCase() ===`
`49`	`49`	`'true';`
`50`	`50`	`}`
`51`	`51`
`@@ -192,7 +192,7 @@ export class AuthService {`
`192`	`192`	`throw new UnauthorizedException('Invalid credentials');`
`193`	`193`	`}`
`194`	`194`
`195`		`- if (!user.isEmailConfirmed) {`
	`195`	`+ if (!user.isEmailConfirmed && this.isMailEnabled) {`
`196`	`196`	`throw new Error('Email not confirmed. Please check your inbox.');`
`197`	`197`	`}`
`198`	`198`