arno/etesync-android
1
0
Fork 0
mirror of https://github.com/etesync/android synced 2024-12-23 15:18:14 +00:00
Code Issues Releases Wiki Activity

Revert "BaseManager: Consider SSLHandshakeExceptions as real issues."

Browse Source 
This change was mostly flagging non-real issues, like connection reset
by peer. Revert for now and come up with an alternative solution.

This reverts commit d3ad17e0bb.
This commit is contained in:
Tom Hacohen 2019-09-09 08:45:08 +01:00
parent 37a1c06dbd
commit 1bbf00c6bd
1 changed files with 0 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
app/src/main/java/com/etesync/syncadapter/journalmanager/BaseManager.kt
View File

@ -7,7 +7,6 @@ import java.io.ByteArrayOutputStream
import java.io.IOException import java.io.IOException
import java.net.HttpURLConnection import java.net.HttpURLConnection
import java.util.logging.Level import java.util.logging.Level
import javax.net.ssl.SSLHandshakeException
abstract class BaseManager { abstract class BaseManager {
@ -22,10 +21,6 @@ abstract class BaseManager {
response = client!!.newCall(request).execute() response = client!!.newCall(request).execute()
} catch (e: IOException) { } catch (e: IOException) {
Logger.log.log(Level.SEVERE, "Failed while connecting to server", e) Logger.log.log(Level.SEVERE, "Failed while connecting to server", e)
if (e is SSLHandshakeException) {
// We don't want to ignore SSLHandshake issues like we do normal IO exceptions
throw e
}
throw Exceptions.ServiceUnavailableException("[" + e.javaClass.name + "] " + e.localizedMessage) throw Exceptions.ServiceUnavailableException("[" + e.javaClass.name + "] " + e.localizedMessage)
} }