Skip to content

feat: jh vuln command#32

Open
thelonewolf1603 wants to merge 6 commits intomainfrom
nd/jh-scan
Open

feat: jh vuln command#32
thelonewolf1603 wants to merge 6 commits intomainfrom
nd/jh-scan

Conversation

@thelonewolf1603
Copy link
Copy Markdown
Contributor

@thelonewolf1603 thelonewolf1603 commented Mar 20, 2026

@thelonewolf1603 thelonewolf1603 changed the title feat: jh scan command feat: jh vuln command Mar 20, 2026
Comment thread vuln.go Outdated
return fmt.Sprintf("\033]8;;%s\033\\%s\033]8;;\033\\", url, v.AdvisoryID)
}

func printAdvisory(v *PackageVulnerability, showRanges bool, verbose bool) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

showRanges is not used

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

removed

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

its still there

Comment thread vuln.go
// topSeverity returns the highest CVSS_V3 score string, or the first available, or "N/A".
func topSeverity(scores []SeverityScore) string {
for _, s := range scores {
if strings.HasPrefix(s.Type, "CVSS_V3") && s.Score != "" {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this doesn't seem to return highest but the first

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants