diff --git a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/Fingerprint.kt b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/Fingerprint.kt index 2e61475..19b8deb 100644 --- a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/Fingerprint.kt +++ b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/Fingerprint.kt @@ -2,10 +2,13 @@ package com.xuqm.sdk.bugcollect object Fingerprint { // exceptionType = exception class name (e.g. "NullPointerException"), NOT the level string + // Only the innermost "at" frame (throw site) is used for grouping. + // Kotlin coroutine dispatcher frames (frame 2+) vary across runs of the same logical error, + // which caused identical errors to create separate issues. fun compute(exceptionType: String, message: String, stack: String): String { - val top3 = stack.lines().filter { it.trim().startsWith("at ") }.take(3).joinToString("|") + val top1 = stack.lines().filter { it.trim().startsWith("at ") }.take(1).joinToString("|") val normalized = message.replace(Regex("\\b\\d{4,}\\b"), "N") - return sha256("$exceptionType:$normalized:$top3") + return sha256("$exceptionType:$normalized:$top1") } private fun sha256(s: String): String = diff --git a/sdk-webview/src/main/java/com/xuqm/sdk/webview/XWebViewView.kt b/sdk-webview/src/main/java/com/xuqm/sdk/webview/XWebViewView.kt index 110dad2..9ff1676 100644 --- a/sdk-webview/src/main/java/com/xuqm/sdk/webview/XWebViewView.kt +++ b/sdk-webview/src/main/java/com/xuqm/sdk/webview/XWebViewView.kt @@ -272,12 +272,21 @@ internal fun storagePermissionsGranted(context: Context, acceptMimes: Array if (cursor.moveToFirst()) cursor.getString(0) else null } + ?: runCatching { + context.contentResolver.query( + uri, arrayOf(android.provider.MediaStore.MediaColumns.DATA), null, null, null + )?.use { cursor -> + if (cursor.moveToFirst()) cursor.getString(0)?.substringAfterLast('/') else null + } + }.getOrNull() - val ext = mimeType?.let { MimeTypeMap.getSingleton().getExtensionFromMimeType(it) } - ?: displayName?.substringAfterLast('.', "").takeIf { !it.isNullOrBlank() } + val ext = displayName?.substringAfterLast('.', "")?.takeIf { it.isNotBlank() } + ?: mimeType?.let { MimeTypeMap.getSingleton().getExtensionFromMimeType(it) } ?: "bin" val base = displayName?.substringBeforeLast('.') ?.replace(Regex("[^a-zA-Z0-9_\\-]"), "_") @@ -464,22 +473,27 @@ fun XWebViewView( return@rememberLauncherForActivityResult } val acceptedMimes = pendingAcceptMimes.value - // ContentResolver returns "application/octet-stream" (or null) for most Office files - // from file-system / Downloads providers. Fall back to extension-based inference so - // .docx / .doc / .xls etc. are not incorrectly rejected. val rawMime = context.contentResolver.getType(uri) - val displayName = context.contentResolver.query( + // Query display name; fall back to MediaStore.DATA file path on devices (e.g. vivo) where + // DISPLAY_NAME is null for content://media/external/file/... URIs. + val displayName: String? = context.contentResolver.query( uri, arrayOf(OpenableColumns.DISPLAY_NAME), null, null, null )?.use { cursor -> if (cursor.moveToFirst()) cursor.getString(0) else null } + ?: runCatching { + context.contentResolver.query( + uri, arrayOf(android.provider.MediaStore.MediaColumns.DATA), null, null, null + )?.use { cursor -> + if (cursor.moveToFirst()) cursor.getString(0)?.substringAfterLast('/') else null + } + }.getOrNull() android.util.Log.d(XWV_FILE, " displayName=$displayName rawMime=$rawMime acceptedMimes=$acceptedMimes") - val actualMime: String? = if (rawMime == null || rawMime == "application/octet-stream") { - val ext = displayName?.substringAfterLast('.', "") - ?.lowercase(Locale.ROOT) - ?.takeIf { it.isNotEmpty() } - val extMime = ext?.let { MimeTypeMap.getSingleton().getMimeTypeFromExtension(it) } - android.util.Log.d(XWV_FILE, " rawMime generic → ext=$ext extMime=$extMime") - extMime ?: rawMime - } else rawMime + val ext = displayName?.substringAfterLast('.', "")?.lowercase(Locale.ROOT)?.takeIf { it.isNotEmpty() } + val extMime = ext?.let { MimeTypeMap.getSingleton().getMimeTypeFromExtension(it) } + android.util.Log.d(XWV_FILE, " ext=$ext extMime=$extMime") + // Prefer extension-based MIME type: ContentResolver can return wrong types for Office files + // on some devices (e.g. vivo reports application/msword for .docx files). + // Fall back to rawMime only when we have no extension information. + val actualMime: String? = extMime ?: rawMime android.util.Log.d(XWV_FILE, " actualMime=$actualMime") val allowed = acceptedMimes.isNullOrEmpty() || acceptedMimes.any { accepted ->