Skip to content

Commit b099611

Browse files
committed
Refactored Auth module
1 parent 58c1d13 commit b099611

File tree

2 files changed

+8
-5
lines changed

2 files changed

+8
-5
lines changed

App/Application/DIContainer/AppDIContainer.swift

+2-3
Original file line numberDiff line numberDiff line change
@@ -41,11 +41,10 @@ final class AppDIContainer {
4141
}
4242
}
4343

44-
// MARK: - Auth protocol conformance to Networking Service
44+
// MARK: - Authentication conformance to Networking Service Protocols
4545
extension AuthNetworkRequest: NetworkCancellable {}
4646
extension AuthNetworkSessionManager: NetworkSessionManager {
4747
public func request(_ request: URLRequest, completion: @escaping CompletionHandler) -> NetworkCancellable {
48-
let request: AuthNetworkRequest = self.request(request, completion: completion)
49-
return request
48+
return authRequest(request, completion: completion)
5049
}
5150
}

DevPods/Authentication/Authentication/Module/AuthNetworkSessionManager.swift

+6-2
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import Alamofire
1212
// NOTE: - We create this wrapper to not expose the use of Alomafire to others (modules and App)
1313
public struct AuthNetworkRequest {
1414
let request: DataRequest
15+
1516
public func cancel() {
1617
request.cancel()
1718
}
@@ -28,7 +29,8 @@ public final class AuthNetworkSessionManager {
2829
sessionManager.retrier = authHandler
2930
}
3031

31-
public func request(_ request: URLRequest, completion: @escaping (Data?, URLResponse?, Error?) -> Void) -> AuthNetworkRequest {
32+
public func authRequest(_ request: URLRequest,
33+
completion: @escaping (Data?, URLResponse?, Error?) -> Void) -> AuthNetworkRequest {
3234

3335
let request = sessionManager.request(request)
3436
.validate(statusCode: 200..<400)
@@ -45,7 +47,9 @@ final class AuthHandler: RequestAdapter, RequestRetrier {
4547
return urlRequest
4648
}
4749

48-
func should(_ manager: SessionManager, retry request: Request, with error: Error, completion: @escaping RequestRetryCompletion) {
50+
func should(_ manager: SessionManager,
51+
retry request: Request, with error: Error,
52+
completion: @escaping RequestRetryCompletion) {
4953
// TODO: - refresh access token if expired using lock or access queue
5054
}
5155
}

0 commit comments

Comments
 (0)