never delete refresh token when refreshing

balland · balland · commit 2530e8f08c3f · 2018-09-14T14:50:30.000+02:00
Otherwise this may cause any error or loss of connectivity leading to refresh token deletion. There is a pending PR on the main repo p2#281 Related to sensational/popscan-android#977
diff --git a/Sources/Flows/OAuth2.swift b/Sources/Flows/OAuth2.swift
@@ -367,7 +367,6 @@ open class OAuth2: OAuth2Base {
 					let data = try response.responseData()
 					let json = try self.parseRefreshTokenResponseData(data)
 					if response.response.statusCode >= 400 {
-						self.clientConfig.refreshToken = nil
 						throw OAuth2Error.generic("Failed with status \(response.response.statusCode)")
 					}
 					self.logger?.debug("OAuth2", msg: "Did use refresh token for access token [\(nil != self.clientConfig.accessToken)]")

Original file line number	Diff line number	Diff line change
`@@ -367,7 +367,6 @@ open class OAuth2: OAuth2Base {`
`367`	`367`	`let data = try response.responseData()`
`368`	`368`	`let json = try self.parseRefreshTokenResponseData(data)`
`369`	`369`	`if response.response.statusCode >= 400 {`
`370`		`- self.clientConfig.refreshToken = nil`
`371`	`370`	`throw OAuth2Error.generic("Failed with status \(response.response.statusCode)")`
`372`	`371`	`}`
`373`	`372`	`self.logger?.debug("OAuth2", msg: "Did use refresh token for access token [\(nil != self.clientConfig.accessToken)]")`