From 141be6e4509f9d922555351cc82a3d906aed2a65 Mon Sep 17 00:00:00 2001
From: Sebastian Schuberth <sebastian@doubleopen.org>
Date: Thu, 15 Feb 2024 16:42:43 +0100
Subject: [PATCH] fix(subversion): Throw `IOException` instead of
 `DownloadException`

The `initWorkingTree()` function is documented to throw `IOException`,
and that is what callers catch. Avoid ORT to crash on a

`UnknownHostException` -> `SVNException` -> `DownloadException`

chain by re-throwing a `IOException` here.

Signed-off-by: Sebastian Schuberth <sebastian@doubleopen.org>
---
 .../subversion/src/main/kotlin/Subversion.kt                  | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/plugins/version-control-systems/subversion/src/main/kotlin/Subversion.kt b/plugins/version-control-systems/subversion/src/main/kotlin/Subversion.kt
index 5341ed2127674..fd1cdcdbc84ac 100644
--- a/plugins/version-control-systems/subversion/src/main/kotlin/Subversion.kt
+++ b/plugins/version-control-systems/subversion/src/main/kotlin/Subversion.kt
@@ -20,13 +20,13 @@
 package org.ossreviewtoolkit.plugins.versioncontrolsystems.subversion
 
 import java.io.File
+import java.io.IOException
 import java.net.InetSocketAddress
 import java.net.URI
 import java.nio.file.Paths
 
 import org.apache.logging.log4j.kotlin.logger
 
-import org.ossreviewtoolkit.downloader.DownloadException
 import org.ossreviewtoolkit.downloader.VersionControlSystem
 import org.ossreviewtoolkit.downloader.WorkingTree
 import org.ossreviewtoolkit.model.VcsInfo
@@ -97,7 +97,7 @@ class Subversion : VersionControlSystem() {
         } catch (e: SVNException) {
             e.showStackTrace()
 
-            throw DownloadException("Unable to initialize a $type working tree in '$targetDir' from ${vcs.url}.", e)
+            throw IOException("Unable to initialize a $type working tree in '$targetDir' from ${vcs.url}.", e)
         }
 
         return getWorkingTree(targetDir)