chore: logging

Author: BFergerson

src/main/kotlin/spp/cli/PlatformCLI.kt

@@ -36,6 +36,7 @@ import org.bouncycastle.openssl.PEMKeyPair
 import org.bouncycastle.openssl.PEMParser
 import org.bouncycastle.openssl.jcajce.JcaPEMKeyConverter
 import org.bouncycastle.util.encoders.Hex
+import org.slf4j.LoggerFactory
 import java.io.File
 import java.io.StringReader
 import java.security.SecureRandom
@@ -50,6 +51,7 @@ import javax.net.ssl.X509TrustManager
 
 object PlatformCLI : CliktCommand(name = "spp-cli", allowMultipleSubcommands = true) {
 
+    private val log = LoggerFactory.getLogger(PlatformCLI::class.java)
     val verbose by option("-v", "--verbose", help = "Enable verbose mode").flag()
     val platformHost: String by option("-p", "--platform", help = "Source++ platform host")
         .default(
@@ -84,6 +86,7 @@ object PlatformCLI : CliktCommand(name = "spp-cli", allowMultipleSubcommands = t
         } else {
             "https://$platformHost"
         }
+        log.info("Connecting to Source++ platform at $serverUrl")
 
         val httpClient = if (certFingerprint != null) {
             OkHttpClient().newBuilder()

src/main/kotlin/spp/cli/commands/developer/instrument/AddBreakpoint.kt

@@ -27,6 +27,7 @@ import com.github.ajalt.clikt.parameters.types.enum
 import com.github.ajalt.clikt.parameters.types.int
 import com.github.ajalt.clikt.parameters.types.long
 import kotlinx.coroutines.runBlocking
+import org.slf4j.LoggerFactory
 import spp.cli.PlatformCLI.apolloClient
 import spp.cli.protocol.instrument.AddLiveBreakpointMutation
 import spp.cli.protocol.instrument.adapter.AddLiveBreakpointMutation_ResponseAdapter.AddLiveBreakpoint
@@ -39,6 +40,7 @@ import spp.protocol.instrument.throttle.ThrottleStep
 
 class AddBreakpoint : CliktCommand(name = "breakpoint", help = "Add a live breakpoint instrument") {
 
+    private val log = LoggerFactory.getLogger(AddBreakpoint::class.java)
     val source by argument(help = "Qualified class name")
     val line by argument(help = "Line number").int()
     val id by option("-id", "-i", help = "Breakpoint identifier")
@@ -62,6 +64,8 @@ class AddBreakpoint : CliktCommand(name = "breakpoint", help = "Add a live break
             ),
             id = Optional.presentIfNotNull(id)
         )
+        log.info("Sending request: {}", input)
+
         val response = try {
             apolloClient.mutation(AddLiveBreakpointMutation(input)).execute()
         } catch (e: Exception) {

src/test/kotlin/integration/BatchLiveInstrumentCLI.kt

@@ -35,6 +35,7 @@ class BatchLiveInstrumentCLI : CLIIntegrationTest() {
         //100 live bps
         val addedLiveBps = mutableListOf<LiveInstrument>()
         for (i in 0..99) {
+            log.info("Adding live breakpoint: bp-$i")
             Main.main(
                 arrayOf(
                     "-v",