优化

app/src/main/java/io/legado/app/lib/webdav/WebDav.kt

@@ -7,9 +7,7 @@ import io.legado.app.exception.NoStackTraceException
 import io.legado.app.help.http.newCallResponse
 import io.legado.app.help.http.okHttpClient
 import io.legado.app.help.http.text
-import io.legado.app.utils.NetworkUtils
-import io.legado.app.utils.printOnDebug
-import io.legado.app.utils.toRequestBody
+import io.legado.app.utils.*
 import kotlinx.coroutines.Dispatchers.IO
 import kotlinx.coroutines.withContext
 import okhttp3.Interceptor
@@ -153,33 +151,32 @@ open class WebDav(val path: String, val authorization: Authorization) {
     private fun parseBody(s: String): List<WebDavFile> {
         val list = ArrayList<WebDavFile>()
         val document = Jsoup.parse(s)
-        val elements = document.getElementsByTag("d:response")
-        httpUrl?.let { urlStr ->
+        val ns = document.findNSPrefix("DAV:")
+        val elements = document.findNS("response", ns)
+        val urlStr = httpUrl ?: return list
         val baseUrl = NetworkUtils.getBaseUrl(urlStr)
         for (element in elements) {
             //依然是优化支持 caddy 自建的 WebDav ，其目录后缀都为“/”, 所以删除“/”的判定，不然无法获取该目录项
-                var href = URLDecoder.decode(element.getElementsByTag("d:href")[0].text(), "UTF-8")
-                if (href.endsWith("/")) {
-                    href = href.removeSuffix("/")
-                }
-                val fileName = href.substring(href.lastIndexOf("/") + 1)
+            val href = URLDecoder.decode(element.findNS("href", ns)[0].text(), "UTF-8")
+                .removeSuffix("/")
+            val fileName = href.substringAfterLast("/")
             val webDavFile: WebDav
             try {
                 val urlName = href.ifEmpty {
                     url.file.replace("/", "")
                 }
                 val contentType = element
-                        .getElementsByTag("d:getcontenttype")
+                    .findNS("getcontenttype", ns)
                     .firstOrNull()?.text().orEmpty()
                 val resourceType = element
-                        .getElementsByTag("d:resourcetype")
+                    .findNS("resourcetype", ns)
                     .firstOrNull()?.html()?.trim().orEmpty()
                 val size = kotlin.runCatching {
-                        element.getElementsByTag("d:getcontentlength")
+                    element.findNS("getcontentlength", ns)
                         .firstOrNull()?.text()?.toLong() ?: 0
                 }.getOrDefault(0)
                 val lastModify: Long = kotlin.runCatching {
-                        element.getElementsByTag("d:getlastmodified")
+                    element.findNS("getlastmodified", ns)
                         .firstOrNull()?.text()?.let {
                             LocalDateTime.parse(it, dateTimeFormatter)
                                 .toInstant(ZoneOffset.of("+8")).toEpochMilli()
@@ -201,7 +198,6 @@ open class WebDav(val path: String, val authorization: Authorization) {
                 e.printOnDebug()
             }
         }
-        }
         return list
     }
 
@@ -380,7 +376,9 @@ open class WebDav(val path: String, val authorization: Authorization) {
             val exception = document.getElementsByTag("s:exception").firstOrNull()?.text()
             val message = document.getElementsByTag("s:message").firstOrNull()?.text()
             if (exception == "ObjectNotFound") {
-                throw ObjectNotFoundException(message ?: "$path doesn't exist. code:${response.code}")
+                throw ObjectNotFoundException(
+                    message ?: "$path doesn't exist. code:${response.code}"
+                )
             }
             throw WebDavException(message ?: "未知错误 code:${response.code}")
         }

app/src/main/java/io/legado/app/utils/JsoupExtensions.kt

@@ -5,6 +5,7 @@ import org.jsoup.nodes.CDataNode
 import org.jsoup.nodes.Element
 import org.jsoup.nodes.Node
 import org.jsoup.nodes.TextNode
+import org.jsoup.select.Elements
 import org.jsoup.select.NodeTraversor
 import org.jsoup.select.NodeVisitor
 
@@ -37,6 +38,20 @@ fun Element.textArray(): Array<String> {
     return text.splitNotBlank("\n")
 }
 
+fun Element.findNS(tag: String, namespace: HashSet<String>): Elements {
+    return select("*|$tag").filter { el ->
+        namespace.contains(el.tagName().substringBefore(":"))
+    }.toElements()
+}
+
+fun Element.findNSPrefix(namespaceURI: String): HashSet<String> {
+    return select("[^xmlns]").map { element ->
+        element.attributes().filter { it.value == namespaceURI }.map { it.key.substring(6) }
+    }.flatten().toHashSet()
+}
+
+fun List<Element>.toElements() = Elements(this)
+
 private fun appendNormalisedText(sb: StringBuilder, textNode: TextNode) {
     val text = textNode.wholeText
     if (preserveWhitespace(textNode.parentNode()) || textNode is CDataNode)