Skip to content

fix(admin): PostgreSQL 세션 시간 저장 수정 - #593

Merged
SHcommit merged 1 commit into
developfrom
fix/592-admin-session-timestamp
Oct 3, 2026
Merged

SHcommit merged 1 commit into
developfrom
fix/592-admin-session-timestamp

Conversation

@SHcommit

@SHcommit SHcommit commented Oct 3, 2026

Copy link
Copy Markdown
Member

요약

  • 관리자 로그인 후 PostgreSQL refresh session 저장이 java.time.Instant JDBC 타입 추론 오류로 실패하던 문제를 수정합니다.
  • 세션 생성·회전·폐기의 timestamptz 파라미터를 UTC OffsetDateTime으로 변환합니다.
  • 실제 PostgreSQL에서 세션 생성·회전·폐기를 검증하는 회귀 테스트를 추가합니다.

Closes #592

원인

PostgreSQL JDBC 42.7.13은 PreparedStatement.setObject(Instant)의 SQL 타입을 추론하지 못합니다. 비밀번호 검증과 last_login_at 갱신 뒤 세션 저장에서 예외가 발생했지만, 로그인 controller가 이를 401로 변환해 자격 증명 오류처럼 보였습니다.

검증

  • RED: JdbcAdminSessionStoreTests.savesAndReloadsSessionTimestampsInPostgres가 Can't infer the SQL type ... java.time.Instant로 실패함을 확인
  • GREEN: 관리자 세션 생성·회전·폐기 PostgreSQL 테스트 3개 통과
  • 기존 AdminSessionServiceTests 통과
  • Node repository hygiene 테스트 213/213 통과
  • planning input·Odii fixture·git diff --check·branch parser 통과
  • 전체 Java: 522개 중 521개 통과, 2개 skip
    • 유일한 실패: ARM64 로컬에서 amd64 PostGIS 이미지를 에뮬레이션한 기존 thirtyThousandPublishedPlacesStayWithinMapInfoLatencyBudgets p95 381.03ms > 200ms
    • Docker가 architecture mismatch 및 timeout 가능성을 명시했으며, 변경 범위와 무관합니다. amd64 GitHub Actions verify를 병합 gate로 사용합니다.

배포 후 확인

  • 실제 관리자 로그인 200
  • identity_admin_sessions refresh session 생성
  • refresh/me/logout 흐름
  • Spring·Nginx health 및 오류 로그

@coderabbitai

coderabbitai Bot commented Oct 3, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration
  • Configuration used: Repository: YRootLab/OnMaru-backend/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 27569f23-52d7-489c-b00f-4d8700a4ee7f
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@SHcommit
SHcommit merged commit d75e9d8 into develop Oct 3, 2026
7 checks passed
@SHcommit
SHcommit deleted the fix/592-admin-session-timestamp branch October 3, 2026 10:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant