Skip to content

Fix iFlytek scoring authentication dates - #30

Merged
yydounai1234 merged 1 commit into
mainfrom
codex/fix-iflytek-scoring-date
Aug 3, 2026
Merged

Fix iFlytek scoring authentication dates#30
yydounai1234 merged 1 commit into
mainfrom
codex/fix-iflytek-scoring-date

Conversation

@HansonL622

Copy link
Copy Markdown

What changed

  • Format the iFlytek Suntone WebSocket authentication date with a fixed two-digit day of month.
  • Add regression coverage for both single-digit and two-digit calendar days.

Root cause

DateTimeFormatter.RFC_1123_DATE_TIME formats dates such as August 1 as Sat, 1 Aug 2026 .... The Suntone authentication endpoint requires EEE, dd MMM yyyy HH:mm:ss GMT, so requests made on days 1–9 were rejected during the WebSocket handshake with HTTP 403 before scoring began.

Impact

Pronunciation scoring can authenticate correctly on every day of the month. The change does not alter credentials, audio processing, score parsing, or frontend behavior.

Validation

  • ./mvnw --batch-mode --no-transfer-progress -Dtest=IflytekScoringProviderDateTest,QwenRealtimeProviderTest test — 23 tests passed
  • ./mvnw --batch-mode --no-transfer-progress clean verify — 188 tests passed
  • Live Suntone probe with the configured local credentials — successful scoring response
  • Existing backend CI runs ./mvnw --batch-mode --no-transfer-progress clean verify, so the new regression tests are included automatically

@fennoai fennoai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

No issues found in this review.

Verified: ./mvnw -Dtest=IflytekScoringProviderDateTest test and git diff --check 66db5d8a5bb9696ccc8678ce81e6f5b4c8618687...11224112245e6cfce7196452fbb138045fc40de4.

@suerzzh suerzzh self-assigned this Aug 3, 2026

@yydounai1234 yydounai1234 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@yydounai1234
yydounai1234 merged commit db677aa into main Aug 3, 2026
11 checks passed
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.

3 participants