Skip to content

Commit accd9c4

Browse files
Merge pull request #22 from writeas/return-error-on-login-404
Don't return early if we get a 404 on login attempt
2 parents 52ea441 + a671f83 commit accd9c4

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

Sources/WriteFreely/WFClient.swift

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -924,7 +924,11 @@ public class WFClient {
924924
}
925925
} else {
926926
// We didn't get a 200 OK, so return a WFError
927-
guard let error = self.translateWFError(fromServerResponse: data) else { return }
927+
guard let error = self.translateWFError(fromServerResponse: data) else {
928+
// We couldn't generate a WFError from the server response data, so return an unknown error.
929+
completion(.failure(WFError.unknownError))
930+
return
931+
}
928932
completion(.failure(error))
929933
}
930934
}
@@ -1088,7 +1092,8 @@ private extension WFClient {
10881092
print("⛔️ \(error.message)")
10891093
return WFError(rawValue: error.code)
10901094
} catch {
1091-
return nil
1095+
print("⛔️ An unknown error occurred.")
1096+
return WFError.unknownError
10921097
}
10931098
}
10941099
}

Sources/WriteFreely/WFError.swift

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ public enum WFError: Int, Error {
1212
case internalServerError = 500
1313
case badGateway = 502
1414
case serviceUnavailable = 503
15+
case unknownError = -1
1516
}
1617

1718
struct ErrorMessage: Codable {

0 commit comments

Comments
 (0)