Do not use coroutine APIs in the access token interceptor

[MiSikora]

We have this piece of code:

pocket-casts-android/modules/services/servers/src/main/java/au/com/shiftyjelly/pocketcasts/servers/di/ServersModule.kt

Lines 248 to 273 in f354e0b

	@Provides
	@TokenInterceptor
	@Singleton
	internal fun provideTokenInterceptor(tokenHandler: TokenHandler): Interceptor {
	val unauthenticatedEndpoints = setOf("security") // Don't attach a token to these methods because they get the token
	return Interceptor { chain ->
	val original = chain.request()
	if (unauthenticatedEndpoints.contains(original.url.encodedPathSegments.firstOrNull())) {
	chain.proceed(original)
	} else {
	val token = runBlocking { tokenHandler.getAccessToken() }
	return@Interceptor if (token != null) {
	val response = chain.proceed(buildRequestWithToken(original, token))
	if (response.code == HttpURLConnection.HTTP_UNAUTHORIZED) {
	tokenHandler.invalidateAccessToken()
	val newToken = runBlocking { tokenHandler.getAccessToken() }
	chain.proceed(buildRequestWithToken(original, newToken))
	} else {
	response
	}
	} else {
	chain.proceed(original)
	}
	}
	}
	}

Using runBlocking can lead to problems. While we don't have any issues at the moment it is a potential footgun. See: square/okhttp#7164