Automattic / Automattic/wordpress-rs

`.userAuthenticationRequired`, `.clientCertificateRequired`, `.clientCertificateRejected` fall to `GenericError` despite closer variants existing

Open Beginner friendly
#1,505 0 comments 0 reactions 0 assignees View on GitHub
Bug Error Handling Swift
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

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.