feat: 깃허브 oauth 연동 by pswaao88 · Pull Request #15 · Team-StackUp/stackup

pswaao88 · 2026-05-15T09:12:59Z

변경 사항

PR 요약

GitHub OAuth 기반 로그인 흐름을 구현하고, Core Server 인증 토큰 체계를
추가했습니다.
프론트는 GitHub 로그인 URL 발급 API를 호출하고, callback으로 받은
code/state를 백엔드에 전달하면 Core Server가 GitHub token 교환, 사용자
생성/갱신, 자체 access/refresh token 발급까지 처리합니다.

주요 변경 사항

GitHub OAuth 로그인 시작/콜백 API 추가
- POST /api/auth/github
- GET /api/auth/github/callback
OAuth 보안 처리 추가
- CSRF 방지용 state 발급/검증
- PKCE code_verifier, code_challenge 적용
- OAuth state는 DB에 저장 후 callback에서 1회 사용
GitHub API 연동 추가
- GitHub access token 교환
- GitHub 사용자 프로필 조회
- primary verified email 조회
- GitHub access token AES-GCM 암호화 저장
사용자 생성/갱신 로직 추가
- github_id 기준 기존 유저 조회
- 신규 유저 생성
- 기존 유저 프로필/토큰 갱신
- soft delete를 고려한 GitHub ID partial unique 제약 적용
Core Server 토큰 체계 추가
- access token: JWT
- refresh token: 랜덤 opaque token
- refresh token은 HttpOnly Secure SameSite 쿠키로 전달
- DB에는 refresh token 원문 대신 SHA-256 hash 저장
- refresh token rotation/revoke 처리
사용자 프로필 조회 API 추가
- GET /api/users/me
- Bearer access token 기반 인증
인증/인가 에러 응답 개선
- AUTH_INVALID_TOKEN
- AUTH_EXPIRED_TOKEN
- AUTH_REVOKED_TOKEN
- AUTH_GITHUB_OAUTH_FAILED
- ACCESS_DENIED
설정/환경변수 정리
- GitHub OAuth client 정보
- JWT TTL
- refresh token 정책
- OAuth state/PKCE 정책
- GitHub API endpoint/version
- refresh token cookie 설정
Docker/실행 가이드 정리
- backend Dockerfile
- .env.example
- Core Server README

후속 개선 사항

인증/OAuth 실패 로그가 아직 부족해 운영 중 원인 추적을 위해 보강이
필요합니다. 특히 state 검증, GitHub token exchange, GitHub profile/
email 조회, token 암호화, user upsert 단계별 실패 지점을 구분해 남기는
방향으로 개선할 예정입니다. 단, code/access token/refresh token 등 민
감값은 로그에 남기지 않습니다.
DI 코드를 간소화하기 위해 일부 Service/Controller를 record로 전환했
지만, @Configuration처럼 record 전환 시 Spring 제약이 생기는 경우가
있었습니다. record와 class가 섞이면 코드 스타일이 애매해질 수 있어, 이
후에는 일관성을 위해 Spring Bean을 일반 class 기반으로 유지할지도 검토
가 필요합니다.

스프링 부트 4부터는 JacksonJsonMessageConverter으로 변경

확인 못했던 인덱스 제거 한번에 인덱스를 추가하기 위함

record로 진행할시 final이되어 문제 발생해 일반 클래스로 되돌림

vercel · 2026-05-15T09:13:04Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
stackup	Ready	Preview, Comment	May 15, 2026 3:01pm

i3months · 2026-05-15T12:30:26Z

+    private final Clock clock;
+
+    @Autowired
+    public RefreshTokenService(


생성자 하나로 합치고 @Autowired 빼는것도 깔끔할듯합니다
SecureRandom과 Clock은 빈으로 등록하고

테스트용 생성자 제거했습니다.
OAuthService, RefreshTokenService 생성자 하나로 합치고 빈등록해서 해결했습니다.

우선 패키지 만들지 않고 common.config에 배치했고 설정 늘어나면 패키지 분리해서 관리하겠습니다.

i3months · 2026-05-15T12:40:33Z

+@Component
+public class GithubApiClient {
+
+    private final RestClient restClient;


외부 api 호출 시 HTTP Interface를 사용하면 깔끔합니다.
인터페이스만 선언하고 구현체는 프록시가 자동 생성하는 패턴이 있습니다.

외부 GIthub API 호출 부분 인터페이스 기준으로 변경했습니다.
실제 요청 경로 코드에 명확하게 보이도록 Github_OAUTH_BaseUrl 과 각 api path로 분리해 처리했습니다.

i3months · 2026-05-15T12:41:33Z

+    private final RestClient restClient;
+
+    public GithubApiClient(GithubOAuthProperties githubOAuthProperties) {
+        this.restClient = RestClient.builder()


타임아웃 관련 설정이 있을까요?

• 거기까지는 생각하지 못했습니다. 각 상황에 맞게 아래처럼 timeout을 설정 해보겠습니다.

connect-timeout 3s: 연결 자체가 3초 이상 걸리면 네트워크 문제로 보고 빠르게 실패 처리

read-timeout 5s: OAuth/프로필 조회는 응답이 작아 5초 이상 지연되면 재시도하도록 실패 처리

레포 분석용 GitHub API는 파일 조회가 많아질 수 있어 이후 별도 timeout으로 분리

i3months · 2026-05-15T12:42:49Z

+배포 프론트 테스트:
+
+```env
+GITHUB_OAUTH_REDIRECT_URI=https://www.udangtang.site/auth/callback


주소는 수정해야할듯합니다

주소 빠르게 테스트하기 위해 했던 흔적이어서 지우겠습니다.

i3months · 2026-05-15T12:43:08Z

+인프라:
+
+```env
+POSTGRES_HOST=localhost


포트 반영 안되어있네요
로컬이면 상관없긴합니다

빠르게 테스트하려고 했던 흔적이어서 배포환경에 맞게 넣을 수 있도록 README.md, application.yml, .env.example 전부 수정했습니다.

기존에 빠르게 테스트 하기위해 했던 흔적 제거

i3months · 2026-05-15T16:07:35Z

고생하셨습니다 👍

pswaao88 added 23 commits May 13, 2026 15:22

refactor: JacksonJsonMessageConverter로 교체

6588e1e

스프링 부트 4부터는 JacksonJsonMessageConverter으로 변경

feat: Storage 예외 타입 추가

ad9143c

refactor: record 클래스로 controller 코드 단순화

4776886

feat: Github OAuth 설정 추가

a801cb2

feat: 인증 응답 DTO 생성

38c6304

feat: 깃허브 인증 API

59e9426

feat: OAuth state 저장 및 DB 추가

b5f64dc

PKCE state 발급 및 callback 연결

9c32c0d

feat: GGithub OAuth token 교환

9e1558f

feat: Github access token 암호화

809bd1f

feat: Github 사용자 생성 및 갱신 연결

1678664

feat: refresh token 발급

3336c96

feat: refresh token cookie 연결

0750d58

feat: 사용자 프로필 조회 API

a24c5e7

feat: 인증 실패 응답 코드 세분화

55eddd5

feat: 공통 에러코드 및 예외 처리 보완

837e139

chore: 깃허브 연동 관려 환경변수 정리

723e09e

chore: core 서버 Dockerfile

dd833d7

refactor: 나중에 인덱스 한번에 잡기 위해 제거

d033387

확인 못했던 인덱스 제거 한번에 인덱스를 추가하기 위함

refactor: DI 간소화를 위한 record 클래스로 변환

f531114

chore: 깃허브 인증 설정값 환경 변수로 분리

a6c1a97

refactor: 깃허브 인증 로직 설정값 주입으로 사용

88123f2

fix: SecurityConfig record에서 일반 클래스로 복구

4319806

record로 진행할시 final이되어 문제 발생해 일반 클래스로 되돌림

refactor: core서버 38010포트 기준으로 수정

80f7c35

vercel Bot deployed to Preview May 15, 2026 09:45 View deployment

csh1668 mentioned this pull request May 15, 2026

feat: 이력서 업로드/목록/삭제 API 및 분석 트리거 발행 #17

Merged

i3months reviewed May 15, 2026

View reviewed changes

pswaao88 added 2 commits May 15, 2026 23:17

refactor: 테스트생성자 제거 및 빈 등록

1e0da84

refactor: Github 외부 호출 HHTTP Interface로 변경

f11d2f9

chore: Github OAuth URL 설정 base URL 기준으로 분리

68977c9

vercel Bot deployed to Preview May 15, 2026 14:38 View deployment

pswaao88 added 2 commits May 15, 2026 23:49

chore: GitHub 외부 호출 타임아웃 설정 추가

388a7ff

chore: Core 서버 배포 환경 설정 관리

7895ab2

기존에 빠르게 테스트 하기위해 했던 흔적 제거

vercel Bot deployed to Preview May 15, 2026 15:01 View deployment

i3months merged commit 252efac into dev May 15, 2026
2 checks passed

Uh oh!

Conversation

pswaao88 commented May 15, 2026

변경 사항

후속 개선 사항

Uh oh!

vercel Bot commented May 15, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

i3months commented May 15, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

vercel Bot commented May 15, 2026 •

edited

Loading