Skip to content

fix: 검증 실패가 남긴 인증서 행을 지운다 - #115

Merged
yessjun merged 1 commit into
mainfrom
fix/cert-row-on-verification-failure
Sep 7, 2026
Merged

fix: 검증 실패가 남긴 인증서 행을 지운다#115
yessjun merged 1 commit into
mainfrom
fix/cert-row-on-verification-failure

Conversation

@yessjun

@yessjun yessjun commented Sep 7, 2026

Copy link
Copy Markdown
Member

📝 작업내용

  • 검증이 끝내 실패해 도메인이 주차될 때, 그때까지 만들어 둔 미발급 인증서 행을 삭제
  • 검증 시작이 인증서 행을 「갱신 중」으로 올려놓고 실패해도 되돌리지 않아, 발급이 시작조차 안 된 이름을 화면이 발급 중으로 계속 보고하던 상태
  • 도메인 화면과 인증서 화면이 같은 도메인을 다르게 말하던 것이 이 결함의 증상
  • 한 번이라도 발급됐거나 만료일을 가진 행은 보존, 실패 사유는 도메인 행에 남으므로 정보 손실 없음

⭐️ 검증

  • 테스트 1321개 통과, 실패 0
  • 수정을 끄고 먼저 돌려 빨간색을 확인, 인증서 행이 1개 남는 것을 정확히 지목
  • 그 과정에서 테스트 자체의 결함도 발견, 드라이버가 자바 시각 타입을 받지 못해 SQL 이 아예 실행되지 않던 것을 간격 문법으로 교체

💬 리뷰 포인트

  • 이미 그 상태로 남아 있는 기존 행은 이 변경이 건드리지 않음, 배포 뒤 검증 재실행으로 정리 예정

Verification puts the row in RENEWING when it starts and nothing moved it back
when the domain parked FAILED, so the certificate screen reported "issuing" for
a name whose issuance had never begun and never would. The two screens then
disagreed about the same domain, which is how it was found.

Only an unissued row goes: one that was ever ACTIVE, or that carries an expiry,
is history worth keeping. Nothing is lost, because the reason the domain failed
is on the domain row, and verifying again creates a fresh certificate row.

The test was run against the unfixed code first and failed on the row that
survives, so it catches the defect rather than merely passing beside it.
@yessjun
yessjun merged commit 92948ff into main Sep 7, 2026
1 check passed
@yessjun
yessjun deleted the fix/cert-row-on-verification-failure branch September 7, 2026 04:48
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