Skip to content

Commit f6e0ca0

Browse files
committed
logs
1 parent c086694 commit f6e0ca0

File tree

2 files changed

+20
-7
lines changed

2 files changed

+20
-7
lines changed

src/main/kotlin/org/digma/intellij/plugin/ui/jcef/pluginapi/PluginApiResourceHandler.kt

Lines changed: 11 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ class PluginApiResourceHandler(val project: Project) : CefResourceHandler {
3434
override fun processRequest(request: CefRequest, callback: CefCallback): Boolean {
3535
Log.log(logger::trace, "processing request {}, [request id:{}]", request.url, request.identifier)
3636

37+
//todo: we don't support multi part form data yet or file elements in the post data
3738
if ((request.postData?.elementCount ?: -1) > 1) {
3839
Log.log(
3940
logger::warn,
@@ -49,9 +50,6 @@ class PluginApiResourceHandler(val project: Project) : CefResourceHandler {
4950

5051

5152
if (hasFileElements(request.postData)) {
52-
53-
//todo: we don't support multi part form data yet or file elements in the post data
54-
5553
Log.log(
5654
logger::warn,
5755
"encountered file element in post data. it is not supported by Digma plugin api handler, [request id:{}]",
@@ -95,15 +93,21 @@ class PluginApiResourceHandler(val project: Project) : CefResourceHandler {
9593

9694
val apiUrl = URI(requestUrl).toURL()
9795

98-
apiResponse = getCommand(apiUrl)?.let { command ->
99-
Log.log(logger::trace, "executing command {}, [request id:{}]", command, requestId)
100-
command.execute(project, requestId, requestMethod, postData, headers)
96+
val command = getCommand(apiUrl)
97+
if (command == null) {
98+
Log.log(logger::trace, "command not found for url {}, [request id:{}]", requestUrl, requestId)
99+
val error = ErrorData("command not found for url [$requestUrl]")
100+
apiResponse = PluginApiHttpResponse.createErrorResponse(404, error)
101+
return
101102
}
103+
104+
Log.log(logger::trace, "executing command {}, [request id:{}]", command.javaClass.name, requestId)
105+
apiResponse = command.execute(project, requestId, requestMethod, postData, headers)
102106
Log.log(logger::trace, "got api response {}, [request id:{}]", apiResponse, requestId)
103107

104108
} catch (e: Throwable) {
105109
Log.warnWithException(logger, e, "processRequest {} failed, [request id:{}]", requestUrl, requestId)
106-
ErrorReporter.Companion.getInstance().reportError("ApiProxyResourceHandler.processRequest", e)
110+
ErrorReporter.Companion.getInstance().reportError("PluginApiResourceHandler.processRequest", e)
107111

108112
val error = ErrorData("encountered exception in plugin api handler [$e]. please check the logs.")
109113
apiResponse = PluginApiHttpResponse.createErrorResponse(500, error)

src/main/kotlin/org/digma/intellij/plugin/ui/jcef/pluginapi/RecentActivityBadgeCommand.kt

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,13 +2,17 @@ package org.digma.intellij.plugin.ui.jcef.pluginapi
22

33
import com.fasterxml.jackson.annotation.JsonCreator
44
import com.fasterxml.jackson.annotation.JsonProperty
5+
import com.intellij.openapi.diagnostic.Logger
56
import com.intellij.openapi.project.Project
7+
import org.digma.intellij.plugin.log.Log
68
import org.digma.intellij.plugin.ui.jcef.jsonToObject
79
import org.digma.intellij.plugin.ui.recentactivity.RecentActivityToolWindowIconChanger
810
import java.beans.ConstructorProperties
911

1012
class RecentActivityBadgeCommand : Command() {
1113

14+
private val logger = Logger.getInstance(this::class.java)
15+
1216
override fun execute(
1317
project: Project,
1418
requestId: Long,
@@ -18,14 +22,18 @@ class RecentActivityBadgeCommand : Command() {
1822
): PluginApiHttpResponse {
1923

2024
try {
25+
Log.log(logger::trace, "recent activity badge command received request id {}", requestId)
2126
val badgeRequest = postData?.let { jsonToObject(it, BadgeRequest::class.java) }
27+
Log.log(logger::trace, "recent activity badge command received request id {}, badgeRequest {}", requestId, badgeRequest)
2228

2329
if (badgeRequest == null) {
30+
Log.log(logger::warn, "recent activity badge command received request id {}, invalid or missing badge request", requestId)
2431
// Return 400 Bad Request, with a small error body
2532
val error = ErrorData("Invalid or missing badge request")
2633
return PluginApiHttpResponse.createErrorResponse(400, error)
2734
}
2835

36+
Log.log(logger::trace, "recent activity badge command received request id {}, badgeRequest {}, setting recent activity tool window icon", requestId, badgeRequest)
2937
val recentActivityToolWindowIconChanger = RecentActivityToolWindowIconChanger.getInstance(project)
3038
if (badgeRequest.status) {
3139
recentActivityToolWindowIconChanger.showBadge()
@@ -41,6 +49,7 @@ class RecentActivityBadgeCommand : Command() {
4149
contentStream = null
4250
)
4351
} catch (e: Throwable) {
52+
Log.warnWithException(logger, e, "recent activity badge command failed, [request id:{}]", requestId)
4453
val error = ErrorData(e.message ?: e.toString())
4554
return PluginApiHttpResponse.createErrorResponse(500, error)
4655
}

0 commit comments

Comments
 (0)