Skip to content

[WOOMOB-1896] Attach full application logs as a file to Zendesk support tickets #16263

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
AdamGrzybkowski wants to merge 4 commits into
base: trunk
Choose a base branch
from
+328 −61
Open

[WOOMOB-1896] Attach full application logs as a file to Zendesk support tickets #16263

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
0328c27
Attach full application logs as a file to Zendesk support tickets
AdamGrzybkowski Jul 15, 2026
4d20ad7
Guard log upload with timeout and read device logs once
AdamGrzybkowski Jul 15, 2026
5a4f2fa
Make attachment file preparation best-effort
AdamGrzybkowski Jul 17, 2026
5740e33
Attach connectivity diagnostic log when contacting support via AI chat
AdamGrzybkowski Jul 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion ...e/src/main/kotlin/com/woocommerce/android/support/zendesk/ZendeskEnvironmentDataSource.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,9 @@ class ZendeskEnvironmentDataSource @Inject constructor() {
"${selectedSite.hostURL} (${selectedSite.stateLogInformation})"
} ?: unknownHostValue

suspend fun getDeviceLogs() = WooLog.getCurrentLogEntries().joinToString("\n").takeLast(maxLogfileLength)
suspend fun getDeviceLogs() = getFullDeviceLogs().takeLast(maxLogfileLength)

suspend fun getFullDeviceLogs() = WooLog.getCurrentLogEntries().joinToString("\n")

private val SiteModel.hostURL: String
get() = UrlUtils.removeScheme(url)
Expand Down
24 changes: 13 additions & 11 deletions ...mmerce/src/main/kotlin/com/woocommerce/android/support/zendesk/ZendeskTicketRepository.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ import android.os.Parcelable
import com.woocommerce.android.applicationpasswords.IsAppPasswordsSupportedForJetpackSite
import com.woocommerce.android.extensions.formatResult
import com.woocommerce.android.support.help.HelpOrigin
import com.woocommerce.android.support.zendesk.RequestConstants.APPLICATION_LOG_FILENAME
import com.woocommerce.android.support.zendesk.RequestConstants.DIAGNOSTIC_LOG_FILENAME
import com.woocommerce.android.support.zendesk.RequestConstants.requestCreationIdentityNotSetErrorMessage
import com.woocommerce.android.support.zendesk.RequestConstants.requestCreationTimeoutErrorMessage
Expand Down Expand Up @@ -70,11 +71,10 @@ class ZendeskTicketRepository @Inject constructor(

val ssr: String? = selectedSite?.let { fetchSSR(it) }

val attachmentTokens = if (diagnosticLog != null) {
listOfNotNull(uploadDiagnosticLog(diagnosticLog))
} else {
emptyList()
}
val attachmentTokens = listOfNotNull(
diagnosticLog?.let { uploadTextAttachment(DIAGNOSTIC_LOG_FILENAME, it) },
uploadTextAttachment(APPLICATION_LOG_FILENAME, envDataSource.getFullDeviceLogs())
)
Comment thread
AdamGrzybkowski marked this conversation as resolved.

val requestCallback = object : ZendeskCallback<Request>() {
override fun onSuccess(result: Request?) {
Expand Down Expand Up @@ -130,19 +130,20 @@ class ZendeskTicketRepository @Inject constructor(
return result.model?.formatResult()
}

private suspend fun uploadDiagnosticLog(
diagnosticLog: String
private suspend fun uploadTextAttachment(
fileName: String,
content: String
): String? {
val tempFile = withContext(dispatchers.io) {
File.createTempFile(DIAGNOSTIC_LOG_FILENAME, null).also {
it.writeText(diagnosticLog)
File.createTempFile(fileName, null).also {
it.writeText(content)

@JorgeMucientes JorgeMucientes Jul 15, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could we also make the local file preparation best-effort? File.createTempFile() and writeText() run before the guarded upload, so an I/O failure—for example, low disk space—would exit the flow before the Zendesk ticket is created. Since the application-log upload now runs for every request, an optional attachment could block contacting support entirely.
Wdyt if we catch file-preparation failures, log them, and continue without the attachment token?

}
}

return try {
suspendCancellableCoroutine { continuation ->
zendeskSettings.uploadProvider?.uploadAttachment(
DIAGNOSTIC_LOG_FILENAME,
fileName,
tempFile,
"text/plain",
Comment thread
AdamGrzybkowski marked this conversation as resolved.
Outdated
object : ZendeskCallback<UploadResponse>() {
Expand All @@ -153,7 +154,7 @@ class ZendeskTicketRepository @Inject constructor(
}

override fun onError(error: ErrorResponse?) {
wooLog.e(WooLog.T.SUPPORT, "Failed to upload diagnostic log")
wooLog.e(WooLog.T.SUPPORT, "Failed to upload attachment: $fileName")
if (continuation.isActive) {
continuation.resume(null)
}
Expand Down Expand Up @@ -362,6 +363,7 @@ private object RequestConstants {
const val requestCreationTimeoutErrorMessage = "Request creation timed out"
const val requestCreationIdentityNotSetErrorMessage = "Request creation failed: identity not set"
const val DIAGNOSTIC_LOG_FILENAME = "connectivitytest_log.txt"
const val APPLICATION_LOG_FILENAME = "application_log.txt"
}

private data class ZendeskCustomFieldsParams(
Expand Down
77 changes: 77 additions & 0 deletions WooCommerce/src/test/kotlin/com/woocommerce/android/support/ZendeskTicketRepositoryTest.kt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,6 +26,7 @@ import org.junit.Test
import org.mockito.kotlin.any
import org.mockito.kotlin.argumentCaptor
import org.mockito.kotlin.doReturn
import org.mockito.kotlin.eq
import org.mockito.kotlin.given
import org.mockito.kotlin.mock
import org.mockito.kotlin.stub
Expand All @@ -40,6 +41,8 @@ import org.wordpress.android.fluxc.store.SiteStore
import zendesk.support.CreateRequest
import zendesk.support.Request
import zendesk.support.RequestProvider
import zendesk.support.UploadProvider
import zendesk.support.UploadResponse

@OptIn(ExperimentalCoroutinesApi::class)
internal class ZendeskTicketRepositoryTest : BaseUnitTest() {
Expand Down Expand Up @@ -728,6 +731,79 @@ internal class ZendeskTicketRepositoryTest : BaseUnitTest() {
assertThat(tags).contains(ZendeskTags.jetpackSiteUsingAppPasswords)
}

@Test
fun `when createRequest is called, then the full device logs are uploaded as an attachment and its token is set`() =
testBlocking {
// given
val expectedToken = "attachment-token"
Comment thread
AdamGrzybkowski marked this conversation as resolved.
val uploadResponse = mock<UploadResponse> { on { token } doReturn expectedToken }
val uploadProvider = mock<UploadProvider>()
given(zendeskSettings.uploadProvider).willReturn(uploadProvider)
val uploadCaptor = argumentCaptor<ZendeskCallback<UploadResponse>>()
val requestCaptor = argumentCaptor<CreateRequest>()

// when
val job = launch {
sut.createRequest(
context = mock(),
origin = HelpOrigin.LOGIN_HELP_NOTIFICATION,
ticketType = TicketType.MobileApp,
selectedSite = null,
subject = "subject",
description = "description",
extraTags = emptyList(),
siteAddress = "siteAddress"
).first()
}

// then
verify(uploadProvider).uploadAttachment(
eq("application_log.txt"),
any(),
eq("text/plain"),
uploadCaptor.capture()
)
uploadCaptor.firstValue.onSuccess(uploadResponse)
advanceUntilIdle()
verify(requestProvider).createRequest(requestCaptor.capture(), any())
job.cancel()

assertThat(requestCaptor.firstValue.attachments).containsExactly(expectedToken)
}

@Test
fun `given the log upload fails, when createRequest is called, then the request is still created without attachments`() =
testBlocking {
// given
val uploadProvider = mock<UploadProvider>()
given(zendeskSettings.uploadProvider).willReturn(uploadProvider)
val uploadCaptor = argumentCaptor<ZendeskCallback<UploadResponse>>()
val requestCaptor = argumentCaptor<CreateRequest>()

// when
val job = launch {
sut.createRequest(
context = mock(),
origin = HelpOrigin.LOGIN_HELP_NOTIFICATION,
ticketType = TicketType.MobileApp,
selectedSite = null,
subject = "subject",
description = "description",
extraTags = emptyList(),
siteAddress = "siteAddress"
).first()
}

// then
verify(uploadProvider).uploadAttachment(any(), any(), any(), uploadCaptor.capture())
uploadCaptor.firstValue.onError(mock())
advanceUntilIdle()
verify(requestProvider).createRequest(requestCaptor.capture(), any())
job.cancel()

assertThat(requestCaptor.firstValue.attachments).isNullOrEmpty()
}

private fun createSUT() {
sut = ZendeskTicketRepository(
zendeskSettings = zendeskSettings,
Expand All @@ -744,6 +820,7 @@ internal class ZendeskTicketRepositoryTest : BaseUnitTest() {
on { totalAvailableMemorySize } doReturn "100"
on { deviceLanguage } doReturn "testLanguage"
on { getDeviceLogs() } doReturn "logs"
on { getFullDeviceLogs() } doReturn "full logs"
on { generateVersionName(any()) } doReturn "version"
on { generateNetworkInformation(any()) } doReturn "networkInfo"
on { generateCombinedLogInformationOfSites(any()) } doReturn "sitesInfo"
Expand Down
Loading