changed order of manager's unauthorized notification

quickthyme · quickthyme · commit ab1c033974cc · 2019-11-03T23:25:00.000-06:00
Signed-off-by: James Hall &lt;james.hall@vmlyr.com&gt;
diff --git a/Sources/WebRequest/Manager/WebRequestManager.swift b/Sources/WebRequest/Manager/WebRequestManager.swift
@@ -130,11 +130,11 @@ private extension WebRequestManager {
                 .filter { $0.state == .ready || $0.state == .unauthorized }
 
             guard let session = self.sessionProvider?.current else {
-                notificationCenter.post(UnauthorizedResponseNotification)
                 if let anyRequest = readyRequests.first?.originalRequest {
                     self.fail(request: anyRequest, withStatus: 401)
                 }
                 for wrapper in readyRequests { wrapper.state = .cancelled }
+                notificationCenter.post(UnauthorizedResponseNotification)
                 return
             }
 
@@ -160,11 +160,11 @@ private extension WebRequestManager {
                 return
         }
 
+        try wrapper.originalRequest.completion?(actualResult, wrapper.originalRequest)
+
         if wrapper.state == .unauthorized {
             notificationCenter.post(UnauthorizedResponseNotification)
         }
-
-        try wrapper.originalRequest.completion?(actualResult, wrapper.originalRequest)
     }
 
     func shouldRefresh(since timestamp: TimeInterval) -> Bool {

Original file line number	Diff line number	Diff line change
`@@ -130,11 +130,11 @@ private extension WebRequestManager {`
`130`	`130`	`.filter { $0.state == .ready \|\| $0.state == .unauthorized }`
`131`	`131`
`132`	`132`	`guard let session = self.sessionProvider?.current else {`
`133`		`- notificationCenter.post(UnauthorizedResponseNotification)`
`134`	`133`	`if let anyRequest = readyRequests.first?.originalRequest {`
`135`	`134`	`self.fail(request: anyRequest, withStatus: 401)`
`136`	`135`	`}`
`137`	`136`	`for wrapper in readyRequests { wrapper.state = .cancelled }`
	`137`	`+ notificationCenter.post(UnauthorizedResponseNotification)`
`138`	`138`	`return`
`139`	`139`	`}`
`140`	`140`
`@@ -160,11 +160,11 @@ private extension WebRequestManager {`
`160`	`160`	`return`
`161`	`161`	`}`
`162`	`162`
	`163`	`+ try wrapper.originalRequest.completion?(actualResult, wrapper.originalRequest)`
	`164`	`+`
`163`	`165`	`if wrapper.state == .unauthorized {`
`164`	`166`	`notificationCenter.post(UnauthorizedResponseNotification)`
`165`	`167`	`}`
`166`		`-`
`167`		`- try wrapper.originalRequest.completion?(actualResult, wrapper.originalRequest)`
`168`	`168`	`}`
`169`	`169`
`170`	`170`	`func shouldRefresh(since timestamp: TimeInterval) -> Bool {`