Automattic / Automattic/wordpress-rs
`.userAuthenticationRequired`, `.clientCertificateRequired`, `.clientCertificateRejected` fall to `GenericError` despite closer variants existing
- Dominant language
- Rust
- Stars
- 36
- Forks
- 5
- Avg merge
- 17h 30m
- Merged PRs (30d)
- 43
Description
> Split out from Automattic/wordpress-rs#1497 — *Swift executor URLSession error audit*. Section B — Unchecked URLError codes we could classify better.
>
> File references are to `native/swift/Sources/wordpress-api/SafeRequestExecutor.swift` unless noted; `:NNN` line numbers were verified against `fix/converge-executor-error-classification`.
**Severity: low.** `HttpAuthenticationRequiredError { hostname, method: Option }` already exists with an optional method, so `.userAuthenticationRequired` (authenticating proxies) can map there with `method: nil`. The mTLS codes could take either the auth variants or `InvalidSslError(.genericSslError)` — either beats `GenericError`. Occurrence is limited to authenticating proxies and mTLS-fronted corporate WordPress — rare but real.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start in native/swift/Sources/wordpress-api/SafeRequestExecutor.swift and inspect the handling of .userAuthenticationRequired, .clientCertificateRequired, and .clientCertificateRejected. Use the existing HttpAuthenticationRequiredError and InvalidSslError variants as the reference, then confirm all three codes no longer fall through to GenericError.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- swift
- Domain
- api
- Issue type
- Bug
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Quiet
- Clarity
- Mostly clear
- Newbie friendliness
- 74/100