Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add: logging middleware #135

Open
wants to merge 8 commits into
base: dev
Choose a base branch
from
Open

Add: logging middleware #135

Show file tree
Hide file tree
Changes from 7 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
032c368
Add: logging middleware
LarryKwon Aug 26, 2024
38dc4f7
activate session command
LarryKwon Aug 31, 2024
9edc2fc
add session Command to AuthModule
LarryKwon Aug 31, 2024
43753cf
deploy setting
LarryKwon Aug 31, 2024
96ea722
complete implementing logger
LarryKwon Sep 1, 2024
c9280c0
Merge branch 'dev' into issue/134/otl-nest-response-tset
LarryKwon Sep 1, 2024
33b669d
add session guard
LarryKwon Aug 31, 2024
7643d42
turn off sql logging and make the sessionguard work
LarryKwon Sep 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
10 changes: 7 additions & 3 deletions docker/Dockerfile.server
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,15 +3,19 @@ FROM node:18 AS build
RUN mkdir -p /var/www/otlplus-server
WORKDIR /var/www/otlplus-server

COPY package*.json ./
COPY package.json ./
RUN npm install

COPY . .
RUN npm run client:generate
RUN npm run build

FROM node:18-slim AS server
RUN apt-get update -y && apt-get install -y openssl
FROM node:18 AS server
RUN rm -rf /var/lib/apt/lists/*
RUN apt-get clean
RUN apt-get install -y openssl

# RUN apt-get update -y && apt-get install -y openssl

COPY --from=build /var/www/otlplus-server/ /var/www/otlplus-server/

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드 패치에 대한 간단한 리뷰를 제공하겠습니다.

잠재적인 버그 위험

  1. 패키지 파일 복사: COPY package*.json ./에서 COPY package.json ./으로 변경하였는데, 만약 package-lock.json이 없다면 의도된 대로 작동하지 않을 수 있습니다. 둘 다 복사하도록 유지하는 것이 좋습니다.

  2. apt-get 명령어 순서:

    • apt-get update가 누락되어 있습니다. 아마 모든 레포지토리 정보를 최신 상태로 보장해야 하기 때문에 이 명령어가 필요할 것입니다.
    • apt-get install 전에 apt-get update를 해야 합니다. 코드의 커멘트 처리된 부분에서 이미 존재하던 부분을 주석 해제하면 문제가 해결될 수 있습니다.

개선 제안

  1. 최적화:

    • Dockerfile에서 여러 RUN 명령을 하나로 결합하여 이미지의 레이어 수를 줄이는 것이 좋습니다. 예를 들어, apt-get update, apt-get install, 그리고 apt-get clean을 하나의 RUN 명령으로 묶는 것이 권장됩니다.
    RUN apt-get update -y && \
    apt-get install -y openssl && \
    apt-get clean && \
    rm -rf /var/lib/apt/lists/*

  2. 헬스체크 추가: 서버가 정상적으로 실행되고 있는지를 확인하기 위해 헬스체크를 추가하는 것을 고려해볼 수 있습니다.

  3. 환경 변수 설정: 필요한 환경 변수를 선언하는 것이 좋습니다. 예를 들어, NODE_ENV 등.

이러한 점들을 고려해서 코드를 수정하면 더 안정적이고 최적화된 컨테이너 이미지를 만들 수 있을 것입니다.

Expand Down
Loading
Loading