Skip to content

Commit 4e588bf

Browse files
committed
add telemetry
1 parent 691d131 commit 4e588bf

File tree

2 files changed

+11
-3
lines changed

2 files changed

+11
-3
lines changed

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/NotificationPollingService.kt

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,8 @@ import software.aws.toolkits.core.utils.error
1818
import software.aws.toolkits.core.utils.getLogger
1919
import software.aws.toolkits.jetbrains.core.DefaultRemoteResourceResolverProvider
2020
import software.aws.toolkits.jetbrains.core.RemoteResourceResolverProvider
21+
import software.aws.toolkits.telemetry.Component
22+
import software.aws.toolkits.telemetry.ToolkitTelemetry
2123
import java.io.InputStream
2224
import java.time.Duration
2325

@@ -118,8 +120,15 @@ class NotificationPollingService : Disposable {
118120
request.connection.headerFields["ETag"]?.firstOrNull() ?: ""
119121
}
120122

121-
private fun emitFailureMetric(exception: Exception?) {
122-
// todo: add metric
123+
private fun emitFailureMetric(e: Exception?) {
124+
ToolkitTelemetry.showNotification(
125+
project = null,
126+
component = Component.Filesystem,
127+
id = "",
128+
reason = "Failed to poll for notifications",
129+
success = false,
130+
reasonDesc = "${e?.javaClass?.simpleName ?: "Unknown"}: ${e?.message ?: "No message"}",
131+
)
123132
}
124133

125134
fun addObserver(observer: (Unit) -> Unit) {

plugins/core/jetbrains-community/src/software/aws/toolkits/jetbrains/core/notifications/ProcessNotificationsBase.kt

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,6 @@ class ProcessNotificationsBase {
2020
init {
2121
NotificationPollingService.getInstance().addObserver {
2222
val list = getNotificationsFromFile()
23-
println("here")
2423
}
2524
}
2625

0 commit comments

Comments
 (0)