Skip to content

Commit

Permalink
fix link in Infos tab
Browse files Browse the repository at this point in the history
  • Loading branch information
@adrienpessu
adrienpessu committed Nov 7, 2023
1 parent 38807d3 commit f5ae971
Showing 1 changed file with 50 additions and 24 deletions.
74 changes: 50 additions & 24 deletions src/main/kotlin/com/github/adrienpessu/sarifviewer/toolWindow/SarifViewerWindowFactory.kt
View file
Original file line number Diff line number Diff line change
Expand Up @@ -29,14 +29,14 @@ import java.io.BufferedReader
import java.io.File
import java.io.FileReader
import java.io.FileWriter
import java.net.URI
import java.nio.charset.Charset
import java.nio.file.Path
import javax.swing.*
import javax.swing.event.HyperlinkEvent
import javax.swing.event.HyperlinkListener
import javax.swing.event.TreeSelectionEvent
import javax.swing.event.TreeSelectionListener
import javax.swing.text.html.HTMLDocument
import javax.swing.text.html.HTMLEditorKit
import javax.swing.tree.DefaultMutableTreeNode

Expand Down Expand Up @@ -65,6 +65,7 @@ class SarifViewerWindowFactory : ToolWindowFactory {
private val details = JTabbedPane()
private val splitPane = JSplitPane(JSplitPane.VERTICAL_SPLIT, false, main, details)
private var sarif: SarifSchema210 = SarifSchema210()
private var myList = JTree()
private val refreshButton: JButton = JButton("Refresh")
private val infos = JEditorPane()
private val steps = JEditorPane()
Expand Down Expand Up @@ -115,9 +116,13 @@ class SarifViewerWindowFactory : ToolWindowFactory {

if (repositoryFullName.isNotEmpty() && currentBranch?.name?.isNotEmpty() == true) {
try {
extractSarif(token, repositoryFullName, currentBranch).let {
val map = extractSarif(token, repositoryFullName, currentBranch)
if (map.isEmpty()) {
add(JLabel("No SARIF file found"))
thisLogger().warn("No SARIF file found")
} else {
thisLogger().info("Load result for the repository $repositoryFullName and branch ${currentBranch.name}")
buildContent(it)
buildContent(map, token, repositoryFullName, currentBranch)
}
} catch (e: SarifViewerException) {
add(JLabel(e.message))
Expand All @@ -131,25 +136,15 @@ class SarifViewerWindowFactory : ToolWindowFactory {
}
}

private fun JBPanel<JBPanel<*>>.extractSarif(
token: String,
repositoryFullName: String,
currentBranch: GitLocalBranch
): HashMap<String, MutableList<Leaf>> {
sarif = service.loadSarifFile(token, repositoryFullName, currentBranch.name)
var map = HashMap<String, MutableList<Leaf>>()
if (sarif.runs?.isEmpty() != false) {
add(JLabel("No SARIF file found"))
thisLogger().warn("No SARIF file found")
} else {
map = service.analyseSarif(sarif)
}

return map
}

private fun JBPanel<JBPanel<*>>.buildSkeleton() {
infos.isEditable = false
infos.addHyperlinkListener(object : HyperlinkListener {

Check notice on line 141 in src/main/kotlin/com/github/adrienpessu/sarifviewer/toolWindow/SarifViewerWindowFactory.kt

View workflow job for this annotation

GitHub Actions / Qodana Community for JVM

Object literal can be converted to lambda 

Convert to lambda
override fun hyperlinkUpdate(hle: HyperlinkEvent?) {
if (HyperlinkEvent.EventType.ACTIVATED == hle?.eventType && hle?.description != null) {
Desktop.getDesktop().browse(URI(hle.description))
}
}
})
steps.isEditable = false
steps.addHyperlinkListener(object : HyperlinkListener {

Check notice on line 149 in src/main/kotlin/com/github/adrienpessu/sarifviewer/toolWindow/SarifViewerWindowFactory.kt

View workflow job for this annotation

GitHub Actions / Qodana Community for JVM

Object literal can be converted to lambda 

Convert to lambda
override fun hyperlinkUpdate(hle: HyperlinkEvent?) {
Expand Down Expand Up @@ -184,7 +179,10 @@ class SarifViewerWindowFactory : ToolWindowFactory {
}

private fun buildContent(
map: HashMap<String, MutableList<Leaf>>
map: HashMap<String, MutableList<Leaf>>,
token: String,
repositoryFullName: String,
currentBranch: GitLocalBranch
) {
val root = DefaultMutableTreeNode(project.name)

Expand All @@ -200,12 +198,19 @@ class SarifViewerWindowFactory : ToolWindowFactory {
refreshButton.apply {
addActionListener {
clearJSplitPane()
buildContent(map)
val map = extractSarif(token, repositoryFullName, currentBranch)
if (map.isEmpty()) {
add(JLabel("No SARIF file found"))
thisLogger().warn("No SARIF file found")
} else {
thisLogger().info("Load result for the repository $repositoryFullName and branch ${currentBranch.name}")
buildContent(map, token, repositoryFullName, currentBranch)
}
}
}


val myList = JTree(root)
myList = JTree(root)

myList.isRootVisible = false
val treeSpeedSearch = TreeSpeedSearch(myList)
Expand All @@ -222,8 +227,14 @@ class SarifViewerWindowFactory : ToolWindowFactory {
val leaves = map[e.path.parentPath.lastPathComponent.toString()]
if (!leaves.isNullOrEmpty()) {
val leaf = leaves.first { it.address == e.path.lastPathComponent.toString() }

val githubAlertUrl = leaf.githubAlertUrl.replace("api.", "").replace("repos/", "").replace("code-scanning/alerts", "security/code-scanning")
val githubURL = "<a target=\"_BLANK\" href=\"$githubAlertUrl\">$githubAlertUrl</a>"

infos.contentType = "text/html"

infos.text =
"${leaf.leafName} \n Level: ${leaf.level} \nRule's name: ${leaf.ruleName} \nRule's description ${leaf.ruleDescription} \nLocation ${leaf.location} \nGitHub alert number: ${leaf.githubAlertNumber} \nGitHub alert url ${leaf.githubAlertUrl}\n"
"${leaf.leafName} <br/> Level: ${leaf.level} <br/>Rule's name: ${leaf.ruleName} <br/>Rule's description ${leaf.ruleDescription} <br/>Location ${leaf.location} <br/>GitHub alert number: ${leaf.githubAlertNumber} <br/>GitHub alert url ${githubURL}\n"

steps.read( leaf.steps.joinToString("<br/>") { step ->
"<a href=\"$step\">${step.split("/").last()}</a>"
Expand Down Expand Up @@ -276,9 +287,24 @@ class SarifViewerWindowFactory : ToolWindowFactory {
}

private fun clearJSplitPane() {
myList = JTree()
infos.text = ""
steps.text = ""
details.isVisible = false
}

private fun extractSarif(
token: String,
repositoryFullName: String,
currentBranch: GitLocalBranch
): HashMap<String, MutableList<Leaf>> {
sarif = service.loadSarifFile(token, repositoryFullName, currentBranch.name)
var map = HashMap<String, MutableList<Leaf>>()
if (sarif.runs?.isEmpty() == false) {
map = service.analyseSarif(sarif)
}

return map
}
}
}

0 comments on commit f5ae971

Please sign in to comment.