Fix/ip hash collision by soohyunme · Pull Request #342 · Pseudo-Lab/DevFactory

soohyunme · 2026-02-06T13:20:31Z

📝 PR 타입

🔨 작업 내용

리버스 프록시(Nginx 등) 환경에서 클라이언트의 실제 IP를 올바르게 식별하도록 개선했습니다.
- 문제 상황: 프록시 뒤에서 실행될 때 request.client.host가 프록시 서버의 IP(127.0.0.1 등)만 반환하여 모든 사용자의 ip_hash가 동일하게 기록되는 현상 발생
- 해결 방법: X-Forwarded-For 및 X-Real-IP 헤더를 확인하는 _get_client_ip 헬퍼 함수를 추가하여 실제 클라이언트 IP를 추출하도록 수정
- 적용 범위: 일반 요청 로그(log_request) 및 페이지뷰 로그(log_page_view)

엮인 이슈

Closes #

📸 스크린샷

🧑‍💻 테스트 결과

📅 체크리스트

이해하기 어려운 부분에 주석을 추가했습니다.
관련된 문서를 업데이트했습니다.

…_members

fix(getcloser): redirect to create view if myId is not exist in ready

feat(getcloser): change image when redeemed

Deploy/cert develop

UserChallengeStatus 오류 수정

…status

…tatus refactor(getcloser): unconfirmed members from receiving TEAM_WAITING …

refactor(getcloser): add dummy data

…gain feat(getcloser): implement try again

fix(getcloser): autoincrement id

…ember-challenge refactor(getcloser): get team member challenge answer info

…eset fix(getcloser): disable challenge dummy data

chore: 라이브 데이터 대응

chore: add `DATA_DIR_HOST` env

…-answers fix(getcloser): @typescript-eslint/no-unused-vars

* docs: update project description * docs: update new teammate * docs: evamp team member section and update project images * docs: change teammate info * docs: reformat teammate info * docs: finalize stable HTML table layout for team section * docs: Update README.en.md * docs: refactor project section to slim text-only list and update content

… workflow

feat(platform): setup devfactory homepage and workflow

… workflow

feat: add activities gallery and history page, discord button

feat: add activities main page

Docs/readme update

refactor(platform): update yujin picture

) * refactor: change title * refactor: update line breaks in index.html intro * refactor(platform): standardize section layout and enhance responsive design

HanNayeoniee and others added 30 commits December 19, 2025 00:14

UserChallengeStatus 오류 수정

53ee23d

Merge branch 'main' into fix/userchallenge

7dd7ee1

fix: from_user_id 삭제

ce085c9

fix: 문제 할당 및 판별 로직 수정

97bfa79

fix(getcloser): redirect to create view if myId is not exist in ready…

5c0ffdc

…_members

feat(getcloser): change image when redeemed

5e9986e

Merge pull request #287 from Pseudo-Lab/fix/getcloser/frontend/redirect

7247aa7

fix(getcloser): redirect to create view if myId is not exist in ready

Merge pull request #288 from Pseudo-Lab/feat/getcloser/frontend/redeem

d324a9f

feat(getcloser): change image when redeemed

Merge pull request #293 from Pseudo-Lab/deploy/cert-develop

1cbc8e2

Deploy/cert develop

Merge pull request #265 from Pseudo-Lab/fix/userchallenge

7a08f67

UserChallengeStatus 오류 수정

refactor(getcloser): unconfirmed members from receiving TEAM_WAITING …

f4d0697

…status

Merge pull request #296 from Pseudo-Lab/refactor/getcloser/progress-s…

53bcc64

…tatus refactor(getcloser): unconfirmed members from receiving TEAM_WAITING …

refactor(getcloser): add dummy data

d5cbc75

Merge pull request #297 from Pseudo-Lab/fix/seeddata

f4365ba

refactor(getcloser): add dummy data

feat(getcloser): implement try again

6624cb8

Merge pull request #298 from Pseudo-Lab/feat/getcloser/frontend/try-a…

60de787

…gain feat(getcloser): implement try again

fix(getcloser): autoincrement id

ca1a8d5

Merge pull request #299 from Pseudo-Lab/fix/getcloser/userchallenge-id

d748c2c

fix(getcloser): autoincrement id

refactor(getcloser): get team member challenge answer info

88e7f7e

Merge pull request #300 from Pseudo-Lab/refactor/getcloser/get-team-m…

89d5d26

…ember-challenge refactor(getcloser): get team member challenge answer info

fix(getcloser): disable challenge dummy data

8558e73

Merge pull request #301 from Pseudo-Lab/fix/getcloser/userchallenge_r…

c6baa77

…eset fix(getcloser): disable challenge dummy data

feat(getcloser): short answers

b412f60

chore: add seed data dir config and bind mount

8e4a417

Merge pull request #302 from Pseudo-Lab/chore/seed-data-dir-mount

9db860f

chore: 라이브 데이터 대응

chore: add DATA_DIR_HOST env

a1d57a6

fix(getcloser): @typescript-eslint/no-unused-vars

ab13f28

Merge pull request #304 from Pseudo-Lab/chore/seed-data-dir-mount

4e040c5

chore: add `DATA_DIR_HOST` env

Merge pull request #303 from Pseudo-Lab/feat/getcloser/frontend/short…

2155f17

…-answers fix(getcloser): @typescript-eslint/no-unused-vars

refactor(getcloser): dissolve team

151db9e

soohyunme and others added 22 commits January 24, 2026 21:06

docs: refactor project section to slim text-only list and update content

086f3f2

feat(platform): setup devfactory homepage and oracle cloud deployment…

7aece22

… workflow

Merge pull request #331 from Pseudo-Lab/feat/platform

b65c609

feat(platform): setup devfactory homepage and workflow

docs: Add 2nd GG GetCloser Review

ef5dad2

docs: Add 2nd GG GetCloser Review

9b842d5

Delete docs/results/docs/results directory

7805aaa

chore(cert): update default season

b4eae9d

feat(platform): setup devfactory homepage and oracle cloud deployment…

78f49af

… workflow

docs: update Yujin member profile image

49fc2a5

fix: title view

8025998

feat: add activities gallery and history page

403ddd5

Merge pull request #335 from Pseudo-Lab/feat/platform

4084c9b

feat: add activities gallery and history page, discord button

feat: add activities main page

0abdab3

Merge pull request #336 from Pseudo-Lab/feat/platform

fc4f61d

feat: add activities main page

Merge origin/main into docs/readme-update and update Yujin's photo

32de76a

Merge pull request #334 from Pseudo-Lab/docs/readme-update

436006e

Docs/readme update

refactor(platform): update yujin picture

653517a

Merge pull request #337 from Pseudo-Lab/refactor/update-members

3f1d861

refactor(platform): update yujin picture

refactor: standardize section layout and enhance responsive design (#338

5401fb7

) * refactor: change title * refactor: update line breaks in index.html intro * refactor(platform): standardize section layout and enhance responsive design

docs: Add 11th plan

4f06efb

fix: improve client IP detection for access logging

36c8598

soohyunme requested a review from yesinkim February 6, 2026 13:20

soohyunme added the 수료증 label Feb 6, 2026

github-actions bot added the needs-review label Feb 6, 2026

soohyunme merged commit 87deb0f into deploy/cert-develop Feb 6, 2026
2 checks passed

soohyunme deleted the fix/ip-hash-collision branch February 6, 2026 13:39

github-actions bot added merged and removed needs-review labels Feb 6, 2026

soohyunme mentioned this pull request Feb 6, 2026

fix: improve client IP detection for access logging #343

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix/ip hash collision#342

Fix/ip hash collision#342
soohyunme merged 108 commits intodeploy/cert-developfrom
fix/ip-hash-collision

soohyunme commented Feb 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Conversation

soohyunme commented Feb 6, 2026

📝 PR 타입

🔨 작업 내용

엮인 이슈

📸 스크린샷

🧑‍💻 테스트 결과

📅 체크리스트

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants