Skip to content

Commit 90efaf6

Browse files
authored
Fix SSM checkInstallation always returning true (#2689)
1 parent d0a018f commit 90efaf6

File tree

1 file changed

+1
-5
lines changed

1 file changed

+1
-5
lines changed

jetbrains-core/src/software/aws/toolkits/jetbrains/services/ecs/resources/SessionManagerPluginInstallationVerification.kt

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,6 @@ import com.intellij.execution.configurations.GeneralCommandLine
77
import com.intellij.execution.process.CapturingProcessHandler
88
import com.intellij.openapi.application.runInEdt
99
import com.intellij.openapi.project.Project
10-
import com.intellij.vcs.log.runInEdt
1110
import kotlinx.coroutines.Dispatchers
1211
import kotlinx.coroutines.runBlocking
1312
import software.aws.toolkits.jetbrains.services.ecs.exec.SessionManagerPluginWarning
@@ -16,10 +15,7 @@ object SessionManagerPluginInstallationVerification {
1615
private fun checkInstallation(): Boolean = runBlocking(Dispatchers.IO) {
1716
try {
1817
val process = CapturingProcessHandler(GeneralCommandLine("session-manager-plugin")).runProcess()
19-
if (process.exitCode != 0) {
20-
false
21-
}
22-
true
18+
process.exitCode == 0
2319
} catch (e: Exception) {
2420
false
2521
}

0 commit comments

Comments
 (0)