From ffafd94a1e59ce9994d12b42a1bba74438477f90 Mon Sep 17 00:00:00 2001 From: XuqmGroup Date: Thu, 18 Jun 2026 21:15:55 +0800 Subject: [PATCH] =?UTF-8?q?chore(bugcollect):=20=E6=B8=85=E7=90=86?= =?UTF-8?q?=E8=B0=83=E8=AF=95=E6=97=A5=E5=BF=97=EF=BC=8C=E5=87=86=E5=A4=87?= =?UTF-8?q?=E5=8F=91=E5=B8=83=E6=AD=A3=E5=BC=8F=E7=89=88=E6=9C=AC?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit --- .../main/java/com/xuqm/sdk/bugcollect/BugCollect.kt | 4 ---- .../main/java/com/xuqm/sdk/bugcollect/LogQueue.kt | 6 +----- .../com/xuqm/sdk/bugcollect/internal/LogUploader.kt | 12 ++++-------- 3 files changed, 5 insertions(+), 17 deletions(-) diff --git a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/BugCollect.kt b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/BugCollect.kt index 13ba96a..9b98ab9 100644 --- a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/BugCollect.kt +++ b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/BugCollect.kt @@ -88,12 +88,10 @@ object BugCollect { ) if (!isReady()) { synchronized(pendingErrorsLock) { pendingErrors.add(event) } - android.util.Log.d("BugCollect", "captureError: SDK not ready, queued (${pendingErrors.size} pending)") return } flushPendingErrors() queue().push(event) - android.util.Log.d("BugCollect", "captureError: pushed to queue") } fun captureCrash(error: Throwable, tags: Map = emptyMap()) { @@ -154,7 +152,6 @@ object BugCollect { if (crashCaptureStarted) return if (!XuqmSDK.isInitialized()) return crashCaptureStarted = true - android.util.Log.w("BugCollect", "startCrashCapture: registering crash handler") CrashCapture.start( appContext = XuqmSDK.appContext, appKey = XuqmSDK.appKey, @@ -183,7 +180,6 @@ object BugCollect { pendingErrors.clear() list } - android.util.Log.d("BugCollect", "flushPendingErrors: ${events.size} events") events.forEach { queue().push(it) } } diff --git a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/LogQueue.kt b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/LogQueue.kt index ef131ae..89cc229 100644 --- a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/LogQueue.kt +++ b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/LogQueue.kt @@ -111,10 +111,7 @@ internal class LogQueue( private fun flush() { // URL 未就绪时跳过,等下次定时 flush - if (logApiUrl.isBlank()) { - Log.d(TAG, "flush: URL blank, skipping") - return - } + if (logApiUrl.isBlank()) return val issues: List val events: List synchronized(lock) { @@ -124,7 +121,6 @@ internal class LogQueue( pendingIssues.clear() pendingEvents.clear() } - Log.d(TAG, "flush: uploading ${issues.size} issues, ${events.size} events, url=$logApiUrl") if (issues.isNotEmpty()) { scope.launch { runCatching { LogUploader.uploadIssues(logApiUrl, issues) } diff --git a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/internal/LogUploader.kt b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/internal/LogUploader.kt index 9c98320..150277a 100644 --- a/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/internal/LogUploader.kt +++ b/sdk-bugcollect/src/main/java/com/xuqm/sdk/bugcollect/internal/LogUploader.kt @@ -38,7 +38,6 @@ internal object LogUploader { fun uploadIssues(logApiUrl: String, issues: List) { if (issues.isEmpty()) return val url = "${resolveUrl(logApiUrl)}/bugcollect/v1/issues/batch" - Log.w(TAG, "uploadIssues: url=$url, count=${issues.size}, thread=${Thread.currentThread().name}") val envelope = createEnvelope() val events = JSONArray() for (issue in issues) { @@ -79,17 +78,14 @@ internal object LogUploader { } private fun post(url: String, body: String) { - android.util.Log.w(TAG, "POST_START url=$url bodyLen=${body.length}") val request = Request.Builder() .url(url) .post(body.toRequestBody(JSON_MEDIA)) .build() - try { - val response = client.newCall(request).execute() - android.util.Log.w(TAG, "POST_DONE code=${response.code} url=$url") - response.close() - } catch (e: Exception) { - android.util.Log.e(TAG, "POST_FAIL url=$url", e) + client.newCall(request).execute().use { response -> + if (!response.isSuccessful) { + Log.w(TAG, "Upload failed: HTTP ${response.code} url=$url") + } } }